Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WebGPU] Update to latest native headers #8081

Merged
merged 2 commits into from
Feb 9, 2024

Conversation

jrprice
Copy link
Contributor

@jrprice jrprice commented Feb 8, 2024

  • Remove #ifdef for requiredFeature[s]Count

  • Pass nullptr to wgpuCreateInstance

    • Emscripten currently requires this
    • Dawn accepts it too

* Remove #ifdef for `requiredFeature[s]Count`

* Pass nullptr to wgpuCreateInstance
  * Emscripten currently requires this
  * Dawn accepts it too
Copy link
Contributor

@steven-johnson steven-johnson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but I assume this will fail until I update Emscripten and Dawn on the buildbots; I'll get to that now

@jrprice
Copy link
Contributor Author

jrprice commented Feb 9, 2024

The failure on the OSX bot looks like it's due to an older version of Emscripten:

/Users/halidenightly/emsdk/upstream/emscripten/cache/sysroot/include/webgpu/webgpu.h:1206:12: note: 'requiredFeaturesCount' declared here
 1206 |     size_t requiredFeaturesCount;
      |            ^

@steven-johnson
Copy link
Contributor

Yep, just re-updated and two were still lagging. Should be good to go once this build finishes.

@steven-johnson
Copy link
Contributor

Thanks so much for the fix!

@steven-johnson steven-johnson merged commit a3baa5d into halide:main Feb 9, 2024
19 checks passed
@jrprice jrprice deleted the webgpu-header-update branch February 9, 2024 18:42
steven-johnson pushed a commit that referenced this pull request Feb 19, 2024
* [WebGPU] Update to latest native headers

* Remove #ifdef for `requiredFeature[s]Count`

* Pass nullptr to wgpuCreateInstance
  * Emscripten currently requires this
  * Dawn accepts it too

* Use nullptr for another wgpuCreateInstance call
steven-johnson added a commit that referenced this pull request Feb 19, 2024
…elease (#8106)

* Don't require Halide_WebGPU when using wasm (#8063) (#8065)

* Don't require Halide_WebGPU when using wasm (#8063)

* trigger buildbots

* [WebGPU] Update to latest native headers (#8081)

* [WebGPU] Update to latest native headers

* Remove #ifdef for `requiredFeature[s]Count`

* Pass nullptr to wgpuCreateInstance
  * Emscripten currently requires this
  * Dawn accepts it too

* Use nullptr for another wgpuCreateInstance call

---------

Co-authored-by: James Price <jrprice@google.com>
ardier pushed a commit to ardier/Halide-mutation that referenced this pull request Mar 3, 2024
* [WebGPU] Update to latest native headers

* Remove #ifdef for `requiredFeature[s]Count`

* Pass nullptr to wgpuCreateInstance
  * Emscripten currently requires this
  * Dawn accepts it too

* Use nullptr for another wgpuCreateInstance call
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants