[Mesa-dev] [Bug 102274] assertion failure in ir_validate.cpp:240
bugzilla-daemon at freedesktop.org
bugzilla-daemon at freedesktop.org
Thu Aug 17 17:30:57 UTC 2017
https://bugs.freedesktop.org/show_bug.cgi?id=102274
--- Comment #1 from Brian Paul <brianp at vmware.com> ---
I bisected the regression to this commit:
commit e2e2c5abd279df1b3aa99c52b81c9cb48fea35fb
Author: Timothy Arceri <tarceri at itsqueeze.com>
Date: Wed Aug 9 13:34:02 2017 +1000
glsl: calculate number of operands in an expression once
Extra validation is added to ir_validate to make sure this is
always updated to the correct numer of operands, as passes like
lower_instructions modify the instructions directly rather then
generating a new one.
The reduction in time is so small that it is not really
measurable. However callgrind was reporting this function as
being called just under 34 million times while compiling the
Deus Ex shaders (just pre-linking was profiled) with 0.20%
spent in this function.
v2:
- make num_operands a unit8_t
- fix unsigned/signed mismatches
Reviewed-by: Thomas Helland <thomashelland90 at gmail.com>
--
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20170817/f3657faf/attachment.html>
More information about the mesa-dev
mailing list