intel/fs: Don't munge source order of 3-src instructions in opt_algebraic
authorIan Romanick <ian.d.romanick@intel.com>
Thu, 1 Jun 2023 21:21:13 +0000 (14:21 -0700)
committerMarge Bot <emma+marge@anholt.net>
Tue, 6 Jun 2023 06:10:53 +0000 (06:10 +0000)
commit4cc3206218aa9b16939842a67d4a988d8e416d6d
treedbc8871f9ef50ded9b1aa5624c804676887a6414
parent60a00f246b9094a12774bc515ca8a90775920fb5
intel/fs: Don't munge source order of 3-src instructions in opt_algebraic

This only impacts ADD3, so at this point it should not have any
affect. As soon as constants are propagated into ADD3 instructions, it
will be a problem.

The worst part is, the ADD3 instrutions that are broken by the old code
aren't even "progress" on this pass.

Reviewed-by: Matt Turner <mattst88@gmail.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/23262>
src/intel/compiler/brw_fs.cpp