Sitelet https://github.com/gpuweb/gpuweb/pull/2834
Skip to content

Raise maxComputeWorkgroupStorageSize from 16352 to 16384 - #2834

Merged
kainino0x merged 1 commit into
gpuweb:mainfrom
kainino0x:shared-16k
May 16, 2022
Merged

kainino0x merged 1 commit into
gpuweb:mainfrom
kainino0x:shared-16k

Conversation

@kainino0x

@kainino0x kainino0x commented May 6, 2022 •

Copy link
Copy Markdown
Contributor

16352 was only required for Metal families Apple1/Apple2 which we don't support anymore anyway.

Issue: #2832


Preview | Diff

16352 was only required for Metal families Apple1/Apple2 which we don't
support anymore anyway.
@kainino0x kainino0x added for webgpu editors meeting tacit resolution queue Editors have agreed and intend to land if no feedback is given labels May 6, 2022
@kainino0x kainino0x added this to the V1.0 milestone May 6, 2022
@kainino0x
kainino0x requested a review from toji May 6, 2022 20:55
@github-actions

github-actions Bot commented May 6, 2022

Copy link
Copy Markdown
Contributor

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

@toji toji left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Yaass! Get those extra 32 bytes!

@kainino0x
kainino0x merged commit 16c7f3b into gpuweb:main May 16, 2022
github-actions Bot added a commit that referenced this pull request May 16, 2022
SHA: 16c7f3b
Reason: push, by @kainino0x

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions Bot added a commit that referenced this pull request May 16, 2022
SHA: 16c7f3b
Reason: push, by @kainino0x

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions Bot added a commit that referenced this pull request May 16, 2022
SHA: 16c7f3b
Reason: push, by @kainino0x

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@kainino0x
kainino0x deleted the shared-16k branch May 16, 2022 23:08
@kainino0x kainino0x removed the tacit resolution queue Editors have agreed and intend to land if no feedback is given label May 16, 2022
@kainino0x kainino0x added the needs-cts-issue This change requires tests (or would need tests if accepted), but may not have a CTS issue filed yet label Jun 29, 2022
jdarpinian pushed a commit to jdarpinian/gpuweb that referenced this pull request Aug 12, 2022
16352 was only required for Metal families Apple1/Apple2 which we don't
support anymore anyway.
@lokokung lokokung removed the needs-cts-issue This change requires tests (or would need tests if accepted), but may not have a CTS issue filed yet label Oct 13, 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.

3 participants