Closed
Bug 1355321
Opened 8 years ago
Closed 8 years ago
[ANGLE] (Follow up Bug 1347866)Remove the dependency of SH_UNROLL_FOR_LOOP_WITH_SAMPLER_ARRAY_INDEX
Categories
(Core :: Graphics: CanvasWebGL, enhancement, P3)
Core
Graphics: CanvasWebGL
Tracking
()
RESOLVED
FIXED
mozilla55
Tracking | Status | |
---|---|---|
firefox55 | --- | fixed |
People
(Reporter: cleu, Assigned: cleu)
References
(Depends on 1 open bug)
Details
(Whiteboard: [gfx-noted])
Attachments
(3 files)
ANGLE has dropped SH_UNROLL_FOR_LOOP_WITH_SAMPLER_ARRAY_INDEX in this changeset.
https://github.com/google/angle/commit/1b896c62934be40cf8a37dc28fabc15590c89a5d
However, we need this flag to avoid webgl conformance failures under Linux, so we have to revert this changeset in Bug1347866.
We should remove this dependency for better integration with newer version of ANGLE.
I have tested ANGLE without this flag on gecko, besides test fails on Linux, we also got minor performance regression on Windows and Mac when running EpicZenGarden.
I think we need to figure out how chromium not depend on this flag to run everything correctly.
Assignee | ||
Updated•8 years ago
|
Assignee | ||
Comment 1•8 years ago
|
||
Chrome has no problem running sampler-array-using-loop-index in webgl conformance test without SH_UNROLL_FOR_LOOP_WITH_SAMPLER_ARRAY_INDEX set under Linux.
I suspect there are some validation code in Gecko prevent us to process the shader code correctly.
Updated•8 years ago
|
Whiteboard: [gfx-noted]
Assignee | ||
Updated•8 years ago
|
Priority: -- → P3
Assignee | ||
Comment 2•8 years ago
|
||
OK, it seems that the failure only happen on try server, I'll add a fail-if os=='linux' on the mochitest-manifest and remove the flag.
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → cleu
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment 9•8 years ago
|
||
mozreview-review |
Comment on attachment 8858201 [details]
Bug 1355321 - Part1: Revert "Bug 1347866 - Part2: Revert "translator: remove code related to for-loop unrolling"";
https://reviewboard.mozilla.org/r/130150/#review132884
Attachment #8858201 -
Flags: review?(jgilbert) → review+
Comment 10•8 years ago
|
||
mozreview-review |
Comment on attachment 8858202 [details]
Bug 1355321 - Part2: Remove SH_UNROLL_FOR_LOOP_WITH_SAMPLER_ARRAY_INDEX;
https://reviewboard.mozilla.org/r/130152/#review132886
Attachment #8858202 -
Flags: review?(jgilbert) → review+
Comment 11•8 years ago
|
||
mozreview-review |
Comment on attachment 8858203 [details]
Bug 1355321 - Part3: Add fail-if to prevent orange light on tryserver;
https://reviewboard.mozilla.org/r/130154/#review132888
Attachment #8858203 -
Flags: review?(jgilbert) → review+
Assignee | ||
Updated•8 years ago
|
Keywords: checkin-needed
Comment 12•8 years ago
|
||
Pushed by ihsiao@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/6a54d6dacf9b
Part1: Revert "Bug 1347866 - Part2: Revert "translator: remove code related to for-loop unrolling""; r=jgilbert
https://hg.mozilla.org/integration/autoland/rev/31967bdfb50d
Part2: Remove SH_UNROLL_FOR_LOOP_WITH_SAMPLER_ARRAY_INDEX; r=jgilbert
https://hg.mozilla.org/integration/autoland/rev/a6563c2d3699
Part3: Add fail-if to prevent orange light on tryserver; r=jgilbert
Keywords: checkin-needed
Comment 13•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/31967bdfb50d
https://hg.mozilla.org/mozilla-central/rev/a6563c2d3699
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in
before you can comment on or make changes to this bug.
Description
•