On Friday, February 23, 2018 3:56:05 PM PST Ian Romanick wrote: > From: Ian Romanick <ian.d.roman...@intel.com> > > If the previously seen instruction generates more fields than the new > instruction, still allow CSE to happen. This doesn't do much, but it > also enables a couple more shaders in the next patch. It helped quite a > bit in another change series that I have (at least for now) abandoned. > > No changes on Skylake, Broadwell, Iron Lake or GM45.
Skylake and Broadwell sort of go without saying, they don't use this compiler :) It might be worth leaving some kind of comment (or renaming variables?) in instructions_match() to indicate that 'a' is the generating expression and 'b' is the second instance...since it's no longer exactly symmetrical. Either way, Reviewed-by: Kenneth Graunke <kenn...@whitecape.org>
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
