nir/load_store_vectorize: don't ignore subgroup memory barriers
Not sure why I thought this was correct, but we should consider them for
optimization purposes.
Fixes: ce9205c03b
('nir: add a load/store vectorization pass')
Signed-off-by: Rhys Perry <pendingchaos02@gmail.com>
Reviewed-by: Jason Ekstrand <jason@jlekstrand.net>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/4202>
This commit is contained in:
parent
4d0b8a9d32
commit
f4eb833a12
|
@ -1195,7 +1195,6 @@ handle_barrier(struct vectorize_ctx *ctx, bool *progress, nir_function_impl *imp
|
|||
release = nir_intrinsic_memory_semantics(intrin) & NIR_MEMORY_RELEASE;
|
||||
switch (nir_intrinsic_memory_scope(intrin)) {
|
||||
case NIR_SCOPE_INVOCATION:
|
||||
case NIR_SCOPE_SUBGROUP:
|
||||
/* a barier should never be required for correctness with these scopes */
|
||||
modes = 0;
|
||||
break;
|
||||
|
|
Loading…
Reference in New Issue