diff options
| author | Jason Ekstrand <[email protected]> | 2019-04-17 17:10:18 -0500 |
|---|---|---|
| committer | Jason Ekstrand <[email protected]> | 2019-04-18 17:12:44 +0000 |
| commit | 85c35885b383477ac4f92a11695fc85c6d45c98c (patch) | |
| tree | b9cb1e3ce1c4e484908019ec13d55ed9e7bae288 /src/compiler/nir/tests | |
| parent | eee994e769259887ea17ec653ee1ea277416ea8b (diff) | |
nir: Rework nir_src_as_alu_instr to not take a pointer
Other nir_src_as_* functions just take a nir_src. It's not that much
more memory copying and the constness preserving really isn't worth the
cognitive dissonance.
Reviewed-by: Eric Anholt <[email protected]>
Reviewed-by: Caio Marcelo de Oliveira Filho <[email protected]>
Diffstat (limited to 'src/compiler/nir/tests')
| -rw-r--r-- | src/compiler/nir/tests/vars_tests.cpp | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/src/compiler/nir/tests/vars_tests.cpp b/src/compiler/nir/tests/vars_tests.cpp index 9e0b5d2c0df..d27a9c0fb1d 100644 --- a/src/compiler/nir/tests/vars_tests.cpp +++ b/src/compiler/nir/tests/vars_tests.cpp @@ -509,7 +509,7 @@ TEST_F(nir_copy_prop_vars_test, load_direct_array_deref_on_vector_reuses_previou ASSERT_TRUE(store->src[1].is_ssa); /* NOTE: The ALU instruction is how we get the vec.y. */ - ASSERT_TRUE(nir_src_as_alu_instr(&store->src[1])); + ASSERT_TRUE(nir_src_as_alu_instr(store->src[1])); } TEST_F(nir_copy_prop_vars_test, load_direct_array_deref_on_vector_reuses_previous_copy) @@ -567,7 +567,7 @@ TEST_F(nir_copy_prop_vars_test, load_direct_array_deref_on_vector_gets_reused) nir_intrinsic_instr *store = get_intrinsic(nir_intrinsic_store_deref, 1); ASSERT_TRUE(store->src[1].is_ssa); - ASSERT_TRUE(nir_src_as_alu_instr(&store->src[1])); + ASSERT_TRUE(nir_src_as_alu_instr(store->src[1])); } TEST_F(nir_copy_prop_vars_test, store_load_direct_array_deref_on_vector) @@ -612,7 +612,7 @@ TEST_F(nir_copy_prop_vars_test, store_load_direct_array_deref_on_vector) /* Fourth store will compose first and second store values. */ nir_intrinsic_instr *fourth_store = get_intrinsic(nir_intrinsic_store_deref, 3); ASSERT_TRUE(fourth_store->src[1].is_ssa); - EXPECT_TRUE(nir_src_as_alu_instr(&fourth_store->src[1])); + EXPECT_TRUE(nir_src_as_alu_instr(fourth_store->src[1])); } TEST_F(nir_copy_prop_vars_test, store_load_indirect_array_deref_on_vector) @@ -990,7 +990,7 @@ TEST_F(nir_combine_stores_test, non_overlapping_stores) ASSERT_EQ(nir_intrinsic_write_mask(combined), 0xf); ASSERT_EQ(nir_intrinsic_get_var(combined, 0), out); - nir_alu_instr *vec = nir_src_as_alu_instr(&combined->src[1]); + nir_alu_instr *vec = nir_src_as_alu_instr(combined->src[1]); ASSERT_TRUE(vec); for (int i = 0; i < 4; i++) { nir_intrinsic_instr *load = @@ -1030,7 +1030,7 @@ TEST_F(nir_combine_stores_test, overlapping_stores) ASSERT_EQ(nir_intrinsic_write_mask(combined), 0xf); ASSERT_EQ(nir_intrinsic_get_var(combined, 0), out); - nir_alu_instr *vec = nir_src_as_alu_instr(&combined->src[1]); + nir_alu_instr *vec = nir_src_as_alu_instr(combined->src[1]); ASSERT_TRUE(vec); /* Component x comes from v[0]. */ @@ -1100,7 +1100,7 @@ TEST_F(nir_combine_stores_test, direct_array_derefs) ASSERT_EQ(nir_intrinsic_write_mask(combined), 0xf); ASSERT_EQ(nir_intrinsic_get_var(combined, 0), out); - nir_alu_instr *vec = nir_src_as_alu_instr(&combined->src[1]); + nir_alu_instr *vec = nir_src_as_alu_instr(combined->src[1]); ASSERT_TRUE(vec); /* Component x comes from v[0]. */ |