diff options
| author | Francisco Jerez <[email protected]> | 2016-03-13 19:26:37 -0700 |
|---|---|---|
| committer | Matt Turner <[email protected]> | 2020-03-06 10:20:39 -0800 |
| commit | ab6d7929864b1c80a8de5b7cd58775f02fe1a7ff (patch) | |
| tree | 5cf9480d7e413fa3619bb593b503bf9330253b2d /src/intel/compiler/brw_vec4.cpp | |
| parent | 65080dc8df00d006912ade2d69d4a06c3d4c5e0a (diff) | |
intel/compiler: Pass detailed dependency classes to invalidate_analysis()
Have fun reading through the whole back-end optimizer to verify
whether I've missed any dependency flags -- Or alternatively, just
trust that any mistake here will trigger an assertion failure during
analysis pass validation if it ever poses a problem for the
consistency of any of the analysis passes managed by the framework.
Reviewed-by: Matt Turner <[email protected]>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/4012>
Diffstat (limited to 'src/intel/compiler/brw_vec4.cpp')
| -rw-r--r-- | src/intel/compiler/brw_vec4.cpp | 24 |
1 files changed, 13 insertions, 11 deletions
diff --git a/src/intel/compiler/brw_vec4.cpp b/src/intel/compiler/brw_vec4.cpp index 18676d4c4ca..de2149f416b 100644 --- a/src/intel/compiler/brw_vec4.cpp +++ b/src/intel/compiler/brw_vec4.cpp @@ -496,7 +496,7 @@ vec4_visitor::opt_vector_float() } if (progress) - invalidate_analysis(DEPENDENCY_EVERYTHING); + invalidate_analysis(DEPENDENCY_INSTRUCTIONS); return progress; } @@ -577,7 +577,7 @@ vec4_visitor::opt_reduce_swizzle() } if (progress) - invalidate_analysis(DEPENDENCY_EVERYTHING); + invalidate_analysis(DEPENDENCY_INSTRUCTION_DETAIL); return progress; } @@ -904,7 +904,8 @@ vec4_visitor::opt_algebraic() } if (progress) - invalidate_analysis(DEPENDENCY_EVERYTHING); + invalidate_analysis(DEPENDENCY_INSTRUCTION_DATA_FLOW | + DEPENDENCY_INSTRUCTION_DETAIL); return progress; } @@ -1474,7 +1475,7 @@ vec4_visitor::opt_register_coalesce() } if (progress) - invalidate_analysis(DEPENDENCY_EVERYTHING); + invalidate_analysis(DEPENDENCY_INSTRUCTIONS); return progress; } @@ -1525,7 +1526,7 @@ vec4_visitor::eliminate_find_live_channel() } if (progress) - invalidate_analysis(DEPENDENCY_EVERYTHING); + invalidate_analysis(DEPENDENCY_INSTRUCTION_DETAIL); return progress; } @@ -1601,7 +1602,7 @@ vec4_visitor::split_virtual_grfs() } } } - invalidate_analysis(DEPENDENCY_EVERYTHING); + invalidate_analysis(DEPENDENCY_INSTRUCTION_DETAIL | DEPENDENCY_VARIABLES); } void @@ -1904,7 +1905,7 @@ vec4_visitor::lower_minmax() } if (progress) - invalidate_analysis(DEPENDENCY_EVERYTHING); + invalidate_analysis(DEPENDENCY_INSTRUCTIONS); return progress; } @@ -2040,7 +2041,8 @@ vec4_visitor::fixup_3src_null_dest() } if (progress) - invalidate_analysis(DEPENDENCY_EVERYTHING); + invalidate_analysis(DEPENDENCY_INSTRUCTION_DETAIL | + DEPENDENCY_VARIABLES); } void @@ -2368,7 +2370,7 @@ vec4_visitor::lower_simd_width() } if (progress) - invalidate_analysis(DEPENDENCY_EVERYTHING); + invalidate_analysis(DEPENDENCY_INSTRUCTIONS | DEPENDENCY_VARIABLES); return progress; } @@ -2525,7 +2527,7 @@ vec4_visitor::scalarize_df() } if (progress) - invalidate_analysis(DEPENDENCY_EVERYTHING); + invalidate_analysis(DEPENDENCY_INSTRUCTIONS); return progress; } @@ -2568,7 +2570,7 @@ vec4_visitor::lower_64bit_mad_to_mul_add() } if (progress) - invalidate_analysis(DEPENDENCY_EVERYTHING); + invalidate_analysis(DEPENDENCY_INSTRUCTIONS | DEPENDENCY_VARIABLES); return progress; } |