diff options
| author | Ian Romanick <[email protected]> | 2018-12-17 18:48:17 -0800 |
|---|---|---|
| committer | Ian Romanick <[email protected]> | 2018-12-18 10:47:54 -0800 |
| commit | 29e4b949b45b468c366b9865298391c89ba6642c (patch) | |
| tree | 1597f2cb26a8977baa18b2b23ee4eeb7a05d4961 | |
| parent | 433ca3127a3b94bfe9a513e7c7ce594e09e1359f (diff) | |
Revert "nir/lower_indirect: Bail early if modes == 0"
"There's no point in walking the program if we're never going to
actually lower anything."
Except we might lower compacted local arrays. In that case, modes will
be 0, but there is still lowering to be done.
This reverts commit 7f75cf2a9408b9af562e033ef6c1d1fd15141421.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=109081
Suggested-by: Kenneth Graunke <[email protected]>
Reviewed-by: Jason Ekstrand <[email protected]>
Reviewed-by: Lionel Landwerlin <[email protected]>
Tested-by: Clayton Craft <[email protected]>
Cc: Kenneth Graunke <[email protected]>
| -rw-r--r-- | src/compiler/nir/nir_lower_indirect_derefs.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/src/compiler/nir/nir_lower_indirect_derefs.c b/src/compiler/nir/nir_lower_indirect_derefs.c index adcfaec219f..cf025bb0425 100644 --- a/src/compiler/nir/nir_lower_indirect_derefs.c +++ b/src/compiler/nir/nir_lower_indirect_derefs.c @@ -205,9 +205,6 @@ nir_lower_indirect_derefs(nir_shader *shader, nir_variable_mode modes) { bool progress = false; - if (modes == 0) - return false; - nir_foreach_function(function, shader) { if (function->impl) progress = lower_indirects_impl(function->impl, modes) || progress; |