Skip to content

Conversation

@toji
Copy link
Member

@toji toji commented May 17, 2021

An alternative to #1660 and #1314, suggested here.

Adds a 'forceSoftware' option to GPURequestAdapterOptions and an isSoftware boolean attribute to GPUAdapter.


Preview | Diff

@github-actions
Copy link
Contributor

Previews, as seen when this build job started (32949a4):
WebGPU | IDL
WGSL
Explainer

kainino0x
kainino0x previously approved these changes May 17, 2021
Copy link
Contributor

@kainino0x kainino0x left a comment

Choose a reason for hiding this comment

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

LGTM

@kainino0x kainino0x requested review from kdashg and litherum May 17, 2021 22:25
@kainino0x
Copy link
Contributor

I guess I should hold my "editor" approval until there's agreement.

@kainino0x kainino0x self-requested a review May 17, 2021 22:26
@kainino0x kainino0x dismissed their stale review May 17, 2021 22:26

need agreement

Copy link
Contributor

@kainino0x kainino0x left a comment

Choose a reason for hiding this comment

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

Resolution: accepted

@toji toji force-pushed the force-software branch from 32949a4 to 0d604d1 Compare June 7, 2021 19:46
@toji
Copy link
Member Author

toji commented Jun 7, 2021

Rebased, should pass the checks cleanly now.

@github-actions
Copy link
Contributor

github-actions bot commented Jun 7, 2021

Previews, as seen when this build job started (0d604d1):
WebGPU | IDL
WGSL
Explainer

@toji toji merged commit 1963320 into main Jun 7, 2021
@toji toji deleted the force-software branch June 7, 2021 20:03
blueboxd pushed a commit to blueboxd/chromium-legacy that referenced this pull request Jun 8, 2021
Matches spec changes in gpuweb/gpuweb#1734.
Currently hard-coded to return false for isSoftware and resolve to null
is forceSoftware is specified, since we don't currently support software
adapters.

Bug: 1069302
Change-Id: Ifb30b31dd642127a64855e22af12ebefdd313fa7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2946050
Commit-Queue: Brandon Jones <[email protected]>
Reviewed-by: Kai Ninomiya <[email protected]>
Cr-Commit-Position: refs/heads/master@{#890065}
ben-clayton pushed a commit to ben-clayton/gpuweb that referenced this pull request Sep 6, 2022
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.

5 participants