[webkit-reviews] review requested: [Bug 201240] [WHLSL] Add an analysis to verify that functions like control barriers and derivatives are not called in non-uniform control flow. : [Attachment 378768] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Sep 13 18:32:23 PDT 2019


Robin Morisset <rmorisset at apple.com> has asked	for review:
Bug 201240: [WHLSL] Add an analysis to verify that functions like control
barriers and derivatives are not called in non-uniform control flow.
https://bugs.webkit.org/show_bug.cgi?id=201240

Attachment 378768: Patch

https://bugs.webkit.org/attachment.cgi?id=378768&action=review




--- Comment #23 from Robin Morisset <rmorisset at apple.com> ---
Created attachment 378768

  --> https://bugs.webkit.org/attachment.cgi?id=378768&action=review

Patch

Implemented Saam's suggestion + the analysis now supports comma on the
left-side of assignment.
I don't believe it is a good idea to have them, but
https://bugs.webkit.org/show_bug.cgi?id=201251 introduced them, and dealing
with them is much easier than finding how to avoid generating them in that
other pass.


More information about the webkit-reviews mailing list