[Bug 94477] Too many temporary expressions in shader freeze glLinkProgram
bugzilla-daemon at freedesktop.org
bugzilla-daemon at freedesktop.org
Fri Sep 2 17:51:37 UTC 2016
https://bugs.freedesktop.org/show_bug.cgi?id=94477
--- Comment #8 from Matt Turner <mattst88 at gmail.com> ---
(In reply to Tapani Pälli from comment #6)
> Created attachment 126170 [details] [review]
> fix
>
> This should result in same effect as the first experimentation, I will run
> testing on this and if it works fine then I'll send it to mesa-dev mailing
> list.
That's pretty clever, but unfortunately it cannot work. Cases of dividing by
zero, inf, nan, etc cannot be handled. We don't do this optimization in
opt_algebraic for exactly that reason.
On top of that, the purpose of the test is to check whether you can compile a
huge expression, and the one it uses is hilariously unrepresentative and prone
to abuse by hacks like this one :)
The test has uncovered real flaws in our compiler: GLSL IR's copy propagation
runs in O(n^2), and the channel_expressions pass creates a huge number of
expressions.
Ken and I have spent time recently trying to handle the shader-db regressions
by disabling channel_expressions. It's slow going and other tasks keep taking
priority.
The most direct way forward then, is to improve copy propagate's run time like
I described in comment 3. Maybe you want to take a stab at replacing the linked
lists in the pass with hash tables? I spent some time on it... maybe you'll
have better luck. :)
--
You are receiving this mail because:
You are the QA Contact for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/intel-3d-bugs/attachments/20160902/59af185d/attachment.html>
More information about the intel-3d-bugs
mailing list