intel/brw: Remove two duplicated validate calls in optimizer

The OPT macro will call validate() after each pass, so both cases
removed by this patch are just redundant calls.  Will only affect
Debug builds since in Release builds validation is a no-op.

Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28534>
This commit is contained in:
Caio Oliveira 2024-04-01 11:57:39 -07:00
parent 8a6fe54409
commit 671d216f39

View file

@ -39,8 +39,6 @@ brw_fs_optimize(fs_visitor &s)
s.assign_constant_locations();
OPT(brw_fs_lower_constant_loads);
s.validate();
if (s.compiler->lower_dpas)
OPT(brw_fs_lower_dpas);
@ -157,8 +155,6 @@ brw_fs_optimize(fs_visitor &s)
OPT(brw_fs_lower_uniform_pull_constant_loads);
OPT(brw_fs_lower_find_live_channel);
s.validate();
}
static unsigned