MakotoUwu opened a new pull request, #19771:
URL: https://github.com/apache/tvm/pull/19771

   This splits out the Web/WebGPU runtime-only portion of #19766 into a smaller 
PR, following reviewer feedback that the compiler-side changes should be 
handled separately.
   
   This PR keeps the scope to `web/` runtime code:
   
   - reorder FFI implementation includes before runtime implementation includes 
in the wasm single-translation-unit build, avoiding static initialization 
ordering issues during module startup
   - make `ArrayDecodeStorage` tolerate `f32-to-bf16` records whose payload is 
already native float32-sized, while preserving the existing packed-bf16 
expansion path
   - load large tensor-cache records in chunks to avoid oversized JS-to-wasm 
decode/copy calls
   - unpack `kTVMFFIShape` callback results as JS number arrays so chunked 
tensor views can pass explicit shape tuples
   
   Local validation:
   
   - `npm run lint` from `web/`
   - `npx tsc --noEmit --pretty false` from `web/`
   - `git diff --check`
   
   I could not run the full local `npm run prepwasm && npm run build` path on 
this machine because Emscripten (`emcc`/`emsdk`) is not installed. The earlier 
broad PR had the Apache wasm CI pass before the compiler-side CI failures; this 
PR is intended to let the wasm job validate the runtime-only split 
independently.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to