Mesa (master): nir/tests: Use nir_scoped_memory_barrier() helper
GitLab Mirror
gitlab-mirror at kemper.freedesktop.org
Mon Feb 24 19:49:56 UTC 2020
Module: Mesa
Branch: master
Commit: 6be766336a0ebb556f44765e3d6f6af0e67070a1
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=6be766336a0ebb556f44765e3d6f6af0e67070a1
Author: Caio Marcelo de Oliveira Filho <caio.oliveira at intel.com>
Date: Thu Jan 9 10:05:31 2020 -0800
nir/tests: Use nir_scoped_memory_barrier() helper
Most of the vars tests already had a local helper, so just drop it in
favor of the one in nir_builder. Remaining two tests changed to use
the helper.
The load_store_vectorizer tests were using the specific memory
barriers, but since scoped barriers are also handled, prefer that.
Reviewed-by: Jason Ekstrand <jason at jlekstrand.net>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3913>
---
.../nir/tests/load_store_vectorizer_tests.cpp | 15 ++++-
src/compiler/nir/tests/vars_tests.cpp | 68 ++++++++++++----------
2 files changed, 48 insertions(+), 35 deletions(-)
diff --git a/src/compiler/nir/tests/load_store_vectorizer_tests.cpp b/src/compiler/nir/tests/load_store_vectorizer_tests.cpp
index 0d749debd4c..710a2da6f72 100644
--- a/src/compiler/nir/tests/load_store_vectorizer_tests.cpp
+++ b/src/compiler/nir/tests/load_store_vectorizer_tests.cpp
@@ -748,7 +748,10 @@ TEST_F(nir_load_store_vectorize_test, ssbo_store_large)
TEST_F(nir_load_store_vectorize_test, ubo_load_adjacent_memory_barrier)
{
create_load(nir_var_mem_ubo, 0, 0, 0x1);
- nir_builder_instr_insert(b, &nir_intrinsic_instr_create(b->shader, nir_intrinsic_memory_barrier)->instr);
+
+ nir_scoped_memory_barrier(b, NIR_SCOPE_DEVICE, NIR_MEMORY_ACQ_REL,
+ nir_var_mem_ssbo);
+
create_load(nir_var_mem_ubo, 0, 4, 0x2);
nir_validate_shader(b->shader, NULL);
@@ -762,7 +765,10 @@ TEST_F(nir_load_store_vectorize_test, ubo_load_adjacent_memory_barrier)
TEST_F(nir_load_store_vectorize_test, ssbo_load_adjacent_memory_barrier)
{
create_load(nir_var_mem_ssbo, 0, 0, 0x1);
- nir_builder_instr_insert(b, &nir_intrinsic_instr_create(b->shader, nir_intrinsic_memory_barrier)->instr);
+
+ nir_scoped_memory_barrier(b, NIR_SCOPE_DEVICE, NIR_MEMORY_ACQ_REL,
+ nir_var_mem_ssbo);
+
create_load(nir_var_mem_ssbo, 0, 4, 0x2);
nir_validate_shader(b->shader, NULL);
@@ -793,7 +799,10 @@ TEST_F(nir_load_store_vectorize_test, ssbo_load_adjacent_barrier)
TEST_F(nir_load_store_vectorize_test, ssbo_load_adjacent_memory_barrier_shared)
{
create_load(nir_var_mem_ssbo, 0, 0, 0x1);
- nir_builder_instr_insert(b, &nir_intrinsic_instr_create(b->shader, nir_intrinsic_memory_barrier_shared)->instr);
+
+ nir_scoped_memory_barrier(b, NIR_SCOPE_WORKGROUP, NIR_MEMORY_ACQ_REL,
+ nir_var_mem_shared);
+
create_load(nir_var_mem_ssbo, 0, 4, 0x2);
nir_validate_shader(b->shader, NULL);
diff --git a/src/compiler/nir/tests/vars_tests.cpp b/src/compiler/nir/tests/vars_tests.cpp
index e660b86ebe6..b9301cea047 100644
--- a/src/compiler/nir/tests/vars_tests.cpp
+++ b/src/compiler/nir/tests/vars_tests.cpp
@@ -195,20 +195,6 @@ class nir_dead_write_vars_test : public nir_vars_test {};
class nir_combine_stores_test : public nir_vars_test {};
class nir_split_vars_test : public nir_vars_test {};
-void
-scoped_memory_barrier(nir_builder *b,
- nir_memory_semantics semantics,
- nir_variable_mode modes,
- nir_scope scope = NIR_SCOPE_DEVICE)
-{
- nir_intrinsic_instr *intrin =
- nir_intrinsic_instr_create(b->shader, nir_intrinsic_scoped_memory_barrier);
- nir_intrinsic_set_memory_semantics(intrin, semantics);
- nir_intrinsic_set_memory_modes(intrin, modes);
- nir_intrinsic_set_memory_scope(intrin, scope);
- nir_builder_instr_insert(b, &intrin->instr);
-}
-
} // namespace
TEST_F(nir_redundant_load_vars_test, duplicated_load)
@@ -497,7 +483,8 @@ TEST_F(nir_copy_prop_vars_test, memory_barrier_in_two_blocks)
nir_store_var(b, v[2], nir_load_var(b, v[0]), 1);
- nir_builder_instr_insert(b, &nir_intrinsic_instr_create(b->shader, nir_intrinsic_memory_barrier)->instr);
+ nir_scoped_memory_barrier(b, NIR_SCOPE_DEVICE, NIR_MEMORY_ACQ_REL,
+ nir_var_mem_ssbo);
nir_store_var(b, v[3], nir_load_var(b, v[1]), 1);
@@ -516,7 +503,8 @@ TEST_F(nir_redundant_load_vars_test, acquire_barrier_prevents_load_removal)
nir_load_var(b, x[0]);
- scoped_memory_barrier(b, NIR_MEMORY_ACQUIRE, nir_var_mem_ssbo);
+ nir_scoped_memory_barrier(b, NIR_SCOPE_DEVICE, NIR_MEMORY_ACQUIRE,
+ nir_var_mem_ssbo);
nir_load_var(b, x[0]);
@@ -533,7 +521,8 @@ TEST_F(nir_redundant_load_vars_test, acquire_barrier_prevents_same_mode_load_rem
nir_load_var(b, x[0]);
nir_load_var(b, x[1]);
- scoped_memory_barrier(b, NIR_MEMORY_ACQUIRE, nir_var_mem_ssbo);
+ nir_scoped_memory_barrier(b, NIR_SCOPE_DEVICE, NIR_MEMORY_ACQUIRE,
+ nir_var_mem_ssbo);
nir_load_var(b, x[0]);
nir_load_var(b, x[1]);
@@ -554,7 +543,8 @@ TEST_F(nir_redundant_load_vars_test, acquire_barrier_allows_different_mode_load_
nir_load_var(b, y[0]);
nir_load_var(b, y[1]);
- scoped_memory_barrier(b, NIR_MEMORY_ACQUIRE, nir_var_mem_ssbo);
+ nir_scoped_memory_barrier(b, NIR_SCOPE_DEVICE, NIR_MEMORY_ACQUIRE,
+ nir_var_mem_ssbo);
nir_load_var(b, x[0]);
nir_load_var(b, x[1]);
@@ -590,7 +580,8 @@ TEST_F(nir_redundant_load_vars_test, release_barrier_allows_load_removal)
nir_load_var(b, x[0]);
- scoped_memory_barrier(b, NIR_MEMORY_RELEASE, nir_var_mem_ssbo);
+ nir_scoped_memory_barrier(b, NIR_SCOPE_DEVICE, NIR_MEMORY_RELEASE,
+ nir_var_mem_ssbo);
nir_load_var(b, x[0]);
@@ -607,7 +598,8 @@ TEST_F(nir_redundant_load_vars_test, release_barrier_allows_same_mode_load_remov
nir_load_var(b, x[0]);
nir_load_var(b, x[1]);
- scoped_memory_barrier(b, NIR_MEMORY_RELEASE, nir_var_mem_ssbo);
+ nir_scoped_memory_barrier(b, NIR_SCOPE_DEVICE, NIR_MEMORY_RELEASE,
+ nir_var_mem_ssbo);
nir_load_var(b, x[0]);
nir_load_var(b, x[1]);
@@ -628,7 +620,8 @@ TEST_F(nir_redundant_load_vars_test, release_barrier_allows_different_mode_load_
nir_load_var(b, y[0]);
nir_load_var(b, y[1]);
- scoped_memory_barrier(b, NIR_MEMORY_RELEASE, nir_var_mem_ssbo);
+ nir_scoped_memory_barrier(b, NIR_SCOPE_DEVICE, NIR_MEMORY_RELEASE,
+ nir_var_mem_ssbo);
nir_load_var(b, x[0]);
nir_load_var(b, x[1]);
@@ -659,7 +652,8 @@ TEST_F(nir_copy_prop_vars_test, acquire_barrier_prevents_propagation)
nir_store_var(b, x[0], nir_imm_int(b, 10), 1);
- scoped_memory_barrier(b, NIR_MEMORY_ACQUIRE, nir_var_mem_ssbo);
+ nir_scoped_memory_barrier(b, NIR_SCOPE_DEVICE, NIR_MEMORY_ACQUIRE,
+ nir_var_mem_ssbo);
nir_load_var(b, x[0]);
@@ -677,7 +671,8 @@ TEST_F(nir_copy_prop_vars_test, acquire_barrier_prevents_same_mode_propagation)
nir_store_var(b, x[0], nir_imm_int(b, 10), 1);
nir_store_var(b, x[1], nir_imm_int(b, 20), 1);
- scoped_memory_barrier(b, NIR_MEMORY_ACQUIRE, nir_var_mem_ssbo);
+ nir_scoped_memory_barrier(b, NIR_SCOPE_DEVICE, NIR_MEMORY_ACQUIRE,
+ nir_var_mem_ssbo);
nir_load_var(b, x[0]);
nir_load_var(b, x[1]);
@@ -699,7 +694,8 @@ TEST_F(nir_copy_prop_vars_test, acquire_barrier_allows_different_mode_propagatio
nir_store_var(b, y[0], nir_imm_int(b, 30), 1);
nir_store_var(b, y[1], nir_imm_int(b, 40), 1);
- scoped_memory_barrier(b, NIR_MEMORY_ACQUIRE, nir_var_mem_ssbo);
+ nir_scoped_memory_barrier(b, NIR_SCOPE_DEVICE, NIR_MEMORY_ACQUIRE,
+ nir_var_mem_ssbo);
nir_load_var(b, x[0]);
nir_load_var(b, x[1]);
@@ -738,7 +734,8 @@ TEST_F(nir_copy_prop_vars_test, release_barrier_allows_propagation)
nir_store_var(b, x[0], nir_imm_int(b, 10), 1);
- scoped_memory_barrier(b, NIR_MEMORY_RELEASE, nir_var_mem_ssbo);
+ nir_scoped_memory_barrier(b, NIR_SCOPE_DEVICE, NIR_MEMORY_RELEASE,
+ nir_var_mem_ssbo);
nir_load_var(b, x[0]);
@@ -755,7 +752,8 @@ TEST_F(nir_copy_prop_vars_test, release_barrier_allows_same_mode_propagation)
nir_store_var(b, x[0], nir_imm_int(b, 10), 1);
nir_store_var(b, x[1], nir_imm_int(b, 20), 1);
- scoped_memory_barrier(b, NIR_MEMORY_RELEASE, nir_var_mem_ssbo);
+ nir_scoped_memory_barrier(b, NIR_SCOPE_DEVICE, NIR_MEMORY_RELEASE,
+ nir_var_mem_ssbo);
nir_load_var(b, x[0]);
nir_load_var(b, x[1]);
@@ -777,7 +775,8 @@ TEST_F(nir_copy_prop_vars_test, release_barrier_allows_different_mode_propagatio
nir_store_var(b, y[0], nir_imm_int(b, 30), 1);
nir_store_var(b, y[1], nir_imm_int(b, 40), 1);
- scoped_memory_barrier(b, NIR_MEMORY_RELEASE, nir_var_mem_ssbo);
+ nir_scoped_memory_barrier(b, NIR_SCOPE_DEVICE, NIR_MEMORY_RELEASE,
+ nir_var_mem_ssbo);
nir_load_var(b, x[0]);
nir_load_var(b, x[1]);
@@ -809,7 +808,8 @@ TEST_F(nir_copy_prop_vars_test, acquire_barrier_prevents_propagation_from_copy)
nir_copy_var(b, x[1], x[0]);
- scoped_memory_barrier(b, NIR_MEMORY_ACQUIRE, nir_var_mem_ssbo);
+ nir_scoped_memory_barrier(b, NIR_SCOPE_DEVICE, NIR_MEMORY_ACQUIRE,
+ nir_var_mem_ssbo);
nir_copy_var(b, x[2], x[1]);
@@ -834,7 +834,8 @@ TEST_F(nir_copy_prop_vars_test, acquire_barrier_prevents_propagation_from_copy_t
nir_copy_var(b, y[0], x[0]);
- scoped_memory_barrier(b, NIR_MEMORY_ACQUIRE, nir_var_mem_ssbo);
+ nir_scoped_memory_barrier(b, NIR_SCOPE_DEVICE, NIR_MEMORY_ACQUIRE,
+ nir_var_mem_ssbo);
nir_copy_var(b, x[1], y[0]);
@@ -858,7 +859,8 @@ TEST_F(nir_copy_prop_vars_test, release_barrier_allows_propagation_from_copy)
nir_copy_var(b, x[1], x[0]);
- scoped_memory_barrier(b, NIR_MEMORY_RELEASE, nir_var_mem_ssbo);
+ nir_scoped_memory_barrier(b, NIR_SCOPE_DEVICE, NIR_MEMORY_RELEASE,
+ nir_var_mem_ssbo);
nir_copy_var(b, x[2], x[1]);
@@ -883,7 +885,8 @@ TEST_F(nir_copy_prop_vars_test, release_barrier_allows_propagation_from_copy_to_
nir_copy_var(b, y[0], x[0]);
- scoped_memory_barrier(b, NIR_MEMORY_RELEASE, nir_var_mem_ssbo);
+ nir_scoped_memory_barrier(b, NIR_SCOPE_DEVICE, NIR_MEMORY_RELEASE,
+ nir_var_mem_ssbo);
nir_copy_var(b, x[1], y[0]);
@@ -1376,7 +1379,8 @@ TEST_F(nir_dead_write_vars_test, DISABLED_memory_barrier_in_two_blocks)
/* Because it is before the barrier, this will kill the previous store to that target. */
nir_store_var(b, v[0], nir_imm_int(b, 3), 1);
- nir_builder_instr_insert(b, &nir_intrinsic_instr_create(b->shader, nir_intrinsic_memory_barrier)->instr);
+ nir_scoped_memory_barrier(b, NIR_SCOPE_DEVICE, NIR_MEMORY_ACQ_REL,
+ nir_var_mem_ssbo);
nir_store_var(b, v[1], nir_imm_int(b, 4), 1);
More information about the mesa-commit
mailing list