summaryrefslogtreecommitdiffstats
path: root/src/intel/Makefile.compiler.am
diff options
context:
space:
mode:
authorIan Romanick <[email protected]>2018-06-20 17:18:30 -0700
committerIan Romanick <[email protected]>2018-12-17 13:47:06 -0800
commit440c051340669e809511c05370d6d703c70f6d0e (patch)
treea469e7857cc69b8aba42cdf420fb76b068b87f4a /src/intel/Makefile.compiler.am
parent111bcc8d028b5d71aacdd080671578b665a9f4ed (diff)
i965/vec4/dce: Don't narrow the write mask if the flags are used
In an instruction sequence like cmp(8).ge.f0.0 vgrf17:D, vgrf2.xxxx:D, vgrf9.xxxx:D (+f0.0) sel(8) vgrf1:UD, vgrf8.xyzw:UD, vgrf1.xyzw:UD The other fields of vgrf17 may be unused, but the CMP still needs to generate the other flag bits. To my surprise, nothing in shader-db or any test suite appears to hit this. However, I have a change to brw_vec4_cmod_propagation that creates cases where this can happen. This fix prevents a couple dozen regressions in that patch. Signed-off-by: Ian Romanick <[email protected]> Reviewed-by: Lionel Landwerlin <[email protected]> Fixes: 5df88c20 ("i965/vec4: Rewrite dead code elimination to use live in/out.")
Diffstat (limited to 'src/intel/Makefile.compiler.am')
-rw-r--r--src/intel/Makefile.compiler.am5
1 files changed, 5 insertions, 0 deletions
diff --git a/src/intel/Makefile.compiler.am b/src/intel/Makefile.compiler.am
index cd7e6882fb9..7c33e35816b 100644
--- a/src/intel/Makefile.compiler.am
+++ b/src/intel/Makefile.compiler.am
@@ -64,6 +64,7 @@ COMPILER_TESTS = \
compiler/test_vf_float_conversions \
compiler/test_vec4_cmod_propagation \
compiler/test_vec4_copy_propagation \
+ compiler/test_vec4_dead_code_eliminate \
compiler/test_vec4_register_coalesce
TESTS += $(COMPILER_TESTS)
@@ -97,6 +98,10 @@ compiler_test_vec4_cmod_propagation_SOURCES = \
compiler/test_vec4_cmod_propagation.cpp
compiler_test_vec4_cmod_propagation_LDADD = $(TEST_LIBS)
+compiler_test_vec4_dead_code_eliminate_SOURCES = \
+ compiler/test_vec4_dead_code_eliminate.cpp
+compiler_test_vec4_dead_code_eliminate_LDADD = $(TEST_LIBS)
+
# Strictly speaking this is neither a C++ test nor using gtest - we can address
# address that at a later point. Until then, this allows us a to simplify things.
compiler_test_eu_compact_SOURCES = \