Ben Clayton | 1773535 | 2021-04-16 09:26:14 +0000 | [diff] [blame] | 1 | // Copyright 2021 The Tint Authors. |
| 2 | // |
| 3 | // Licensed under the Apache License, Version 2.0 (the "License"); |
| 4 | // you may not use this file except in compliance with the License. |
| 5 | // You may obtain a copy of the License at |
| 6 | // |
| 7 | // http://www.apache.org/licenses/LICENSE-2.0 |
| 8 | // |
| 9 | // Unless required by applicable law or agreed to in writing, software |
| 10 | // distributed under the License is distributed on an "AS IS" BASIS, |
| 11 | // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| 12 | // See the License for the specific language governing permissions and |
| 13 | // limitations under the License. |
| 14 | |
| 15 | #include "src/resolver/resolver.h" |
| 16 | |
| 17 | #include "gmock/gmock.h" |
Ben Clayton | a996ffb | 2022-02-02 23:07:11 +0000 | [diff] [blame] | 18 | #include "src/ast/struct_block_attribute.h" |
Ben Clayton | 1773535 | 2021-04-16 09:26:14 +0000 | [diff] [blame] | 19 | #include "src/resolver/resolver_test_helper.h" |
Antonio Maiorano | 5cd71b8 | 2021-04-16 19:07:51 +0000 | [diff] [blame] | 20 | #include "src/sem/struct.h" |
Ben Clayton | 1773535 | 2021-04-16 09:26:14 +0000 | [diff] [blame] | 21 | |
| 22 | namespace tint { |
| 23 | namespace resolver { |
| 24 | namespace { |
| 25 | |
| 26 | using ResolverStorageClassValidationTest = ResolverTest; |
| 27 | |
| 28 | TEST_F(ResolverStorageClassValidationTest, GlobalVariableNoStorageClass_Fail) { |
| 29 | // var g : f32; |
| 30 | Global(Source{{12, 34}}, "g", ty.f32(), ast::StorageClass::kNone); |
| 31 | |
| 32 | EXPECT_FALSE(r()->Resolve()); |
| 33 | EXPECT_EQ(r()->error(), |
Ben Clayton | 5a88ec8 | 2021-06-29 14:42:19 +0000 | [diff] [blame] | 34 | "12:34 error: global variables must have a storage class"); |
Ben Clayton | 1773535 | 2021-04-16 09:26:14 +0000 | [diff] [blame] | 35 | } |
| 36 | |
James Price | 13c79be | 2022-01-12 20:51:36 +0000 | [diff] [blame] | 37 | TEST_F(ResolverStorageClassValidationTest, |
| 38 | GlobalVariableFunctionStorageClass_Fail) { |
| 39 | // var<function> g : f32; |
| 40 | Global(Source{{12, 34}}, "g", ty.f32(), ast::StorageClass::kFunction); |
| 41 | |
| 42 | EXPECT_FALSE(r()->Resolve()); |
| 43 | EXPECT_EQ(r()->error(), |
| 44 | "12:34 error: variables declared at module scope must not be in " |
| 45 | "the function storage class"); |
| 46 | } |
| 47 | |
James Price | db2f5fc | 2022-01-12 20:51:36 +0000 | [diff] [blame] | 48 | TEST_F(ResolverStorageClassValidationTest, Private_RuntimeArray) { |
| 49 | Global(Source{{12, 34}}, "v", ty.array(ty.i32()), |
| 50 | ast::StorageClass::kPrivate); |
| 51 | |
| 52 | EXPECT_FALSE(r()->Resolve()); |
| 53 | EXPECT_EQ( |
| 54 | r()->error(), |
| 55 | R"(12:34 error: runtime-sized arrays can only be used in the <storage> storage class |
| 56 | 12:34 note: while instantiating variable v)"); |
| 57 | } |
| 58 | |
| 59 | TEST_F(ResolverStorageClassValidationTest, Private_RuntimeArrayInStruct) { |
| 60 | auto* s = Structure("S", {Member("m", ty.array(ty.i32()))}, {StructBlock()}); |
| 61 | Global(Source{{12, 34}}, "v", ty.Of(s), ast::StorageClass::kPrivate); |
| 62 | |
| 63 | EXPECT_FALSE(r()->Resolve()); |
| 64 | EXPECT_EQ( |
| 65 | r()->error(), |
| 66 | R"(12:34 error: runtime-sized arrays can only be used in the <storage> storage class |
| 67 | note: while analysing structure member S.m |
| 68 | 12:34 note: while instantiating variable v)"); |
| 69 | } |
| 70 | |
| 71 | TEST_F(ResolverStorageClassValidationTest, Workgroup_RuntimeArray) { |
| 72 | Global(Source{{12, 34}}, "v", ty.array(ty.i32()), |
| 73 | ast::StorageClass::kWorkgroup); |
| 74 | |
| 75 | EXPECT_FALSE(r()->Resolve()); |
| 76 | EXPECT_EQ( |
| 77 | r()->error(), |
| 78 | R"(12:34 error: runtime-sized arrays can only be used in the <storage> storage class |
| 79 | 12:34 note: while instantiating variable v)"); |
| 80 | } |
| 81 | |
| 82 | TEST_F(ResolverStorageClassValidationTest, Workgroup_RuntimeArrayInStruct) { |
| 83 | auto* s = Structure("S", {Member("m", ty.array(ty.i32()))}, {StructBlock()}); |
| 84 | Global(Source{{12, 34}}, "v", ty.Of(s), ast::StorageClass::kWorkgroup); |
| 85 | |
| 86 | EXPECT_FALSE(r()->Resolve()); |
| 87 | EXPECT_EQ( |
| 88 | r()->error(), |
| 89 | R"(12:34 error: runtime-sized arrays can only be used in the <storage> storage class |
| 90 | note: while analysing structure member S.m |
| 91 | 12:34 note: while instantiating variable v)"); |
| 92 | } |
| 93 | |
Ben Clayton | 85bfea6 | 2021-04-19 20:20:33 +0000 | [diff] [blame] | 94 | TEST_F(ResolverStorageClassValidationTest, StorageBufferBool) { |
James Price | 7395e29 | 2022-01-19 15:55:56 +0000 | [diff] [blame] | 95 | // var<storage> g : bool; |
| 96 | Global(Source{{56, 78}}, "g", ty.bool_(), ast::StorageClass::kStorage, |
Ben Clayton | a996ffb | 2022-02-02 23:07:11 +0000 | [diff] [blame] | 97 | ast::AttributeList{ |
| 98 | create<ast::BindingAttribute>(0), |
| 99 | create<ast::GroupAttribute>(0), |
James Price | 7395e29 | 2022-01-19 15:55:56 +0000 | [diff] [blame] | 100 | }); |
| 101 | |
| 102 | ASSERT_FALSE(r()->Resolve()); |
| 103 | |
| 104 | EXPECT_EQ( |
| 105 | r()->error(), |
| 106 | R"(56:78 error: Type 'bool' cannot be used in storage class 'storage' as it is non-host-shareable |
| 107 | 56:78 note: while instantiating variable g)"); |
| 108 | } |
| 109 | |
| 110 | TEST_F(ResolverStorageClassValidationTest, StorageBufferPointer) { |
| 111 | // var<storage> g : ptr<private, f32>; |
| 112 | Global(Source{{56, 78}}, "g", |
| 113 | ty.pointer(ty.f32(), ast::StorageClass::kPrivate), |
| 114 | ast::StorageClass::kStorage, |
Ben Clayton | a996ffb | 2022-02-02 23:07:11 +0000 | [diff] [blame] | 115 | ast::AttributeList{ |
| 116 | create<ast::BindingAttribute>(0), |
| 117 | create<ast::GroupAttribute>(0), |
James Price | 7395e29 | 2022-01-19 15:55:56 +0000 | [diff] [blame] | 118 | }); |
| 119 | |
| 120 | ASSERT_FALSE(r()->Resolve()); |
| 121 | |
| 122 | EXPECT_EQ( |
| 123 | r()->error(), |
| 124 | R"(56:78 error: Type 'ptr<private, f32, read_write>' cannot be used in storage class 'storage' as it is non-host-shareable |
| 125 | 56:78 note: while instantiating variable g)"); |
| 126 | } |
| 127 | |
| 128 | TEST_F(ResolverStorageClassValidationTest, StorageBufferIntScalar) { |
Ben Clayton | 3f968e7 | 2021-05-10 19:16:46 +0000 | [diff] [blame] | 129 | // var<storage> g : i32; |
Ben Clayton | 620d77e | 2021-06-04 19:55:08 +0000 | [diff] [blame] | 130 | Global(Source{{56, 78}}, "g", ty.i32(), ast::StorageClass::kStorage, |
Ben Clayton | a996ffb | 2022-02-02 23:07:11 +0000 | [diff] [blame] | 131 | ast::AttributeList{ |
| 132 | create<ast::BindingAttribute>(0), |
| 133 | create<ast::GroupAttribute>(0), |
Ben Clayton | 3f968e7 | 2021-05-10 19:16:46 +0000 | [diff] [blame] | 134 | }); |
Ben Clayton | 1773535 | 2021-04-16 09:26:14 +0000 | [diff] [blame] | 135 | |
James Price | 7395e29 | 2022-01-19 15:55:56 +0000 | [diff] [blame] | 136 | ASSERT_TRUE(r()->Resolve()) << r()->error(); |
Ben Clayton | 1773535 | 2021-04-16 09:26:14 +0000 | [diff] [blame] | 137 | } |
| 138 | |
James Price | 7395e29 | 2022-01-19 15:55:56 +0000 | [diff] [blame] | 139 | TEST_F(ResolverStorageClassValidationTest, StorageBufferVector) { |
James Price | 14c0b8a | 2021-06-24 15:53:26 +0000 | [diff] [blame] | 140 | // var<storage> g : vec4<f32>; |
| 141 | Global(Source{{56, 78}}, "g", ty.vec4<f32>(), ast::StorageClass::kStorage, |
Ben Clayton | a996ffb | 2022-02-02 23:07:11 +0000 | [diff] [blame] | 142 | ast::AttributeList{ |
| 143 | create<ast::BindingAttribute>(0), |
| 144 | create<ast::GroupAttribute>(0), |
Ben Clayton | 3f968e7 | 2021-05-10 19:16:46 +0000 | [diff] [blame] | 145 | }); |
Ben Clayton | 1773535 | 2021-04-16 09:26:14 +0000 | [diff] [blame] | 146 | |
James Price | 7395e29 | 2022-01-19 15:55:56 +0000 | [diff] [blame] | 147 | ASSERT_TRUE(r()->Resolve()) << r()->error(); |
Ben Clayton | 1773535 | 2021-04-16 09:26:14 +0000 | [diff] [blame] | 148 | } |
| 149 | |
Ben Clayton | 85bfea6 | 2021-04-19 20:20:33 +0000 | [diff] [blame] | 150 | TEST_F(ResolverStorageClassValidationTest, StorageBufferArray) { |
Ben Clayton | 93e8f52 | 2021-06-04 20:41:47 +0000 | [diff] [blame] | 151 | // var<storage, read> g : array<S, 3>; |
Ben Clayton | ba6ab5e | 2021-05-07 14:49:34 +0000 | [diff] [blame] | 152 | auto* s = Structure("S", {Member("a", ty.f32())}); |
Ben Clayton | 8758f10 | 2021-06-09 14:32:14 +0000 | [diff] [blame] | 153 | auto* a = ty.array(ty.Of(s), 3); |
Ben Clayton | 93e8f52 | 2021-06-04 20:41:47 +0000 | [diff] [blame] | 154 | Global(Source{{56, 78}}, "g", a, ast::StorageClass::kStorage, |
| 155 | ast::Access::kRead, |
Ben Clayton | a996ffb | 2022-02-02 23:07:11 +0000 | [diff] [blame] | 156 | ast::AttributeList{ |
| 157 | create<ast::BindingAttribute>(0), |
| 158 | create<ast::GroupAttribute>(0), |
Ben Clayton | 3f968e7 | 2021-05-10 19:16:46 +0000 | [diff] [blame] | 159 | }); |
Ben Clayton | 1773535 | 2021-04-16 09:26:14 +0000 | [diff] [blame] | 160 | |
James Price | 7395e29 | 2022-01-19 15:55:56 +0000 | [diff] [blame] | 161 | ASSERT_TRUE(r()->Resolve()) << r()->error(); |
Ben Clayton | 1773535 | 2021-04-16 09:26:14 +0000 | [diff] [blame] | 162 | } |
| 163 | |
Ben Clayton | 85bfea6 | 2021-04-19 20:20:33 +0000 | [diff] [blame] | 164 | TEST_F(ResolverStorageClassValidationTest, StorageBufferBoolAlias) { |
Ben Clayton | 1773535 | 2021-04-16 09:26:14 +0000 | [diff] [blame] | 165 | // type a = bool; |
Ben Clayton | 93e8f52 | 2021-06-04 20:41:47 +0000 | [diff] [blame] | 166 | // var<storage, read> g : a; |
Ben Clayton | 6e7d24d | 2021-06-09 14:32:14 +0000 | [diff] [blame] | 167 | auto* a = Alias("a", ty.bool_()); |
Ben Clayton | 8758f10 | 2021-06-09 14:32:14 +0000 | [diff] [blame] | 168 | Global(Source{{56, 78}}, "g", ty.Of(a), ast::StorageClass::kStorage, |
Ben Clayton | a996ffb | 2022-02-02 23:07:11 +0000 | [diff] [blame] | 169 | ast::AttributeList{ |
| 170 | create<ast::BindingAttribute>(0), |
| 171 | create<ast::GroupAttribute>(0), |
Ben Clayton | 3f968e7 | 2021-05-10 19:16:46 +0000 | [diff] [blame] | 172 | }); |
Ben Clayton | 1773535 | 2021-04-16 09:26:14 +0000 | [diff] [blame] | 173 | |
| 174 | ASSERT_FALSE(r()->Resolve()); |
| 175 | |
| 176 | EXPECT_EQ( |
| 177 | r()->error(), |
Ben Clayton | a9156ff | 2021-11-05 16:51:38 +0000 | [diff] [blame] | 178 | R"(56:78 error: Type 'bool' cannot be used in storage class 'storage' as it is non-host-shareable |
| 179 | 56:78 note: while instantiating variable g)"); |
Ben Clayton | 1773535 | 2021-04-16 09:26:14 +0000 | [diff] [blame] | 180 | } |
| 181 | |
Sarah | 085dcbb | 2021-07-28 09:59:35 +0000 | [diff] [blame] | 182 | TEST_F(ResolverStorageClassValidationTest, NotStorage_AccessMode) { |
| 183 | // var<private, read> g : a; |
| 184 | Global(Source{{56, 78}}, "g", ty.i32(), ast::StorageClass::kPrivate, |
| 185 | ast::Access::kRead); |
| 186 | |
| 187 | ASSERT_FALSE(r()->Resolve()); |
| 188 | |
| 189 | EXPECT_EQ( |
| 190 | r()->error(), |
Sarah | 4038fa7 | 2021-08-05 15:18:29 +0000 | [diff] [blame] | 191 | R"(56:78 error: only variables in <storage> storage class may declare an access mode)"); |
Sarah | 085dcbb | 2021-07-28 09:59:35 +0000 | [diff] [blame] | 192 | } |
| 193 | |
Ben Clayton | 85bfea6 | 2021-04-19 20:20:33 +0000 | [diff] [blame] | 194 | TEST_F(ResolverStorageClassValidationTest, StorageBufferNoError_Basic) { |
| 195 | // [[block]] struct S { x : i32 }; |
Ben Clayton | 93e8f52 | 2021-06-04 20:41:47 +0000 | [diff] [blame] | 196 | // var<storage, read> g : S; |
Ben Clayton | ba6ab5e | 2021-05-07 14:49:34 +0000 | [diff] [blame] | 197 | auto* s = Structure("S", {Member(Source{{12, 34}}, "x", ty.i32())}, |
Ben Clayton | a996ffb | 2022-02-02 23:07:11 +0000 | [diff] [blame] | 198 | {create<ast::StructBlockAttribute>()}); |
Ben Clayton | 8758f10 | 2021-06-09 14:32:14 +0000 | [diff] [blame] | 199 | Global(Source{{56, 78}}, "g", ty.Of(s), ast::StorageClass::kStorage, |
Ben Clayton | 93e8f52 | 2021-06-04 20:41:47 +0000 | [diff] [blame] | 200 | ast::Access::kRead, |
Ben Clayton | a996ffb | 2022-02-02 23:07:11 +0000 | [diff] [blame] | 201 | ast::AttributeList{ |
| 202 | create<ast::BindingAttribute>(0), |
| 203 | create<ast::GroupAttribute>(0), |
Ben Clayton | 3f968e7 | 2021-05-10 19:16:46 +0000 | [diff] [blame] | 204 | }); |
Ben Clayton | 1773535 | 2021-04-16 09:26:14 +0000 | [diff] [blame] | 205 | |
| 206 | ASSERT_TRUE(r()->Resolve()); |
| 207 | } |
| 208 | |
Ben Clayton | 85bfea6 | 2021-04-19 20:20:33 +0000 | [diff] [blame] | 209 | TEST_F(ResolverStorageClassValidationTest, StorageBufferNoError_Aliases) { |
| 210 | // [[block]] struct S { x : i32 }; |
Ben Clayton | 1773535 | 2021-04-16 09:26:14 +0000 | [diff] [blame] | 211 | // type a1 = S; |
Ben Clayton | 93e8f52 | 2021-06-04 20:41:47 +0000 | [diff] [blame] | 212 | // var<storage, read> g : a1; |
Ben Clayton | ba6ab5e | 2021-05-07 14:49:34 +0000 | [diff] [blame] | 213 | auto* s = Structure("S", {Member(Source{{12, 34}}, "x", ty.i32())}, |
Ben Clayton | a996ffb | 2022-02-02 23:07:11 +0000 | [diff] [blame] | 214 | {create<ast::StructBlockAttribute>()}); |
Ben Clayton | 8758f10 | 2021-06-09 14:32:14 +0000 | [diff] [blame] | 215 | auto* a1 = Alias("a1", ty.Of(s)); |
| 216 | auto* a2 = Alias("a2", ty.Of(a1)); |
| 217 | Global(Source{{56, 78}}, "g", ty.Of(a2), ast::StorageClass::kStorage, |
Ben Clayton | 93e8f52 | 2021-06-04 20:41:47 +0000 | [diff] [blame] | 218 | ast::Access::kRead, |
Ben Clayton | a996ffb | 2022-02-02 23:07:11 +0000 | [diff] [blame] | 219 | ast::AttributeList{ |
| 220 | create<ast::BindingAttribute>(0), |
| 221 | create<ast::GroupAttribute>(0), |
Ben Clayton | 3f968e7 | 2021-05-10 19:16:46 +0000 | [diff] [blame] | 222 | }); |
Ben Clayton | 1773535 | 2021-04-16 09:26:14 +0000 | [diff] [blame] | 223 | |
| 224 | ASSERT_TRUE(r()->Resolve()); |
| 225 | } |
| 226 | |
Sarah | 52b6a00 | 2021-06-16 19:40:13 +0000 | [diff] [blame] | 227 | TEST_F(ResolverStorageClassValidationTest, UniformBuffer_Struct_Runtime) { |
| 228 | // [[block]] struct S { m: array<f32>; }; |
Ben Clayton | 01e4b6f | 2022-01-19 22:46:57 +0000 | [diff] [blame] | 229 | // @group(0) @binding(0) var<uniform, > svar : S; |
Sarah | 52b6a00 | 2021-06-16 19:40:13 +0000 | [diff] [blame] | 230 | |
| 231 | auto* s = Structure(Source{{12, 34}}, "S", {Member("m", ty.array<i32>())}, |
Ben Clayton | a996ffb | 2022-02-02 23:07:11 +0000 | [diff] [blame] | 232 | {create<ast::StructBlockAttribute>()}); |
Sarah | 52b6a00 | 2021-06-16 19:40:13 +0000 | [diff] [blame] | 233 | |
| 234 | Global(Source{{56, 78}}, "svar", ty.Of(s), ast::StorageClass::kUniform, |
Ben Clayton | a996ffb | 2022-02-02 23:07:11 +0000 | [diff] [blame] | 235 | ast::AttributeList{ |
| 236 | create<ast::BindingAttribute>(0), |
| 237 | create<ast::GroupAttribute>(0), |
Sarah | 52b6a00 | 2021-06-16 19:40:13 +0000 | [diff] [blame] | 238 | }); |
| 239 | |
| 240 | ASSERT_FALSE(r()->Resolve()); |
James Price | db2f5fc | 2022-01-12 20:51:36 +0000 | [diff] [blame] | 241 | EXPECT_EQ( |
| 242 | r()->error(), |
| 243 | R"(56:78 error: runtime-sized arrays can only be used in the <storage> storage class |
| 244 | note: while analysing structure member S.m |
| 245 | 56:78 note: while instantiating variable svar)"); |
Sarah | 52b6a00 | 2021-06-16 19:40:13 +0000 | [diff] [blame] | 246 | } |
Ben Clayton | 4270834 | 2021-04-21 17:55:12 +0000 | [diff] [blame] | 247 | |
| 248 | TEST_F(ResolverStorageClassValidationTest, UniformBufferBool) { |
| 249 | // var<uniform> g : bool; |
Ben Clayton | 3f968e7 | 2021-05-10 19:16:46 +0000 | [diff] [blame] | 250 | Global(Source{{56, 78}}, "g", ty.bool_(), ast::StorageClass::kUniform, |
Ben Clayton | a996ffb | 2022-02-02 23:07:11 +0000 | [diff] [blame] | 251 | ast::AttributeList{ |
| 252 | create<ast::BindingAttribute>(0), |
| 253 | create<ast::GroupAttribute>(0), |
Ben Clayton | 3f968e7 | 2021-05-10 19:16:46 +0000 | [diff] [blame] | 254 | }); |
Ben Clayton | 4270834 | 2021-04-21 17:55:12 +0000 | [diff] [blame] | 255 | |
| 256 | ASSERT_FALSE(r()->Resolve()); |
| 257 | |
| 258 | EXPECT_EQ( |
| 259 | r()->error(), |
Ben Clayton | a9156ff | 2021-11-05 16:51:38 +0000 | [diff] [blame] | 260 | R"(56:78 error: Type 'bool' cannot be used in storage class 'uniform' as it is non-host-shareable |
| 261 | 56:78 note: while instantiating variable g)"); |
Ben Clayton | 4270834 | 2021-04-21 17:55:12 +0000 | [diff] [blame] | 262 | } |
| 263 | |
| 264 | TEST_F(ResolverStorageClassValidationTest, UniformBufferPointer) { |
James Price | 7395e29 | 2022-01-19 15:55:56 +0000 | [diff] [blame] | 265 | // var<uniform> g : ptr<private, f32>; |
| 266 | Global(Source{{56, 78}}, "g", |
| 267 | ty.pointer(ty.f32(), ast::StorageClass::kPrivate), |
| 268 | ast::StorageClass::kUniform, |
Ben Clayton | a996ffb | 2022-02-02 23:07:11 +0000 | [diff] [blame] | 269 | ast::AttributeList{ |
| 270 | create<ast::BindingAttribute>(0), |
| 271 | create<ast::GroupAttribute>(0), |
Ben Clayton | 3f968e7 | 2021-05-10 19:16:46 +0000 | [diff] [blame] | 272 | }); |
Ben Clayton | 4270834 | 2021-04-21 17:55:12 +0000 | [diff] [blame] | 273 | |
| 274 | ASSERT_FALSE(r()->Resolve()); |
| 275 | |
| 276 | EXPECT_EQ( |
| 277 | r()->error(), |
James Price | 7395e29 | 2022-01-19 15:55:56 +0000 | [diff] [blame] | 278 | R"(56:78 error: Type 'ptr<private, f32, read_write>' cannot be used in storage class 'uniform' as it is non-host-shareable |
| 279 | 56:78 note: while instantiating variable g)"); |
| 280 | } |
| 281 | |
| 282 | TEST_F(ResolverStorageClassValidationTest, UniformBufferIntScalar) { |
| 283 | // var<uniform> g : i32; |
| 284 | Global(Source{{56, 78}}, "g", ty.i32(), ast::StorageClass::kUniform, |
Ben Clayton | a996ffb | 2022-02-02 23:07:11 +0000 | [diff] [blame] | 285 | ast::AttributeList{ |
| 286 | create<ast::BindingAttribute>(0), |
| 287 | create<ast::GroupAttribute>(0), |
James Price | 7395e29 | 2022-01-19 15:55:56 +0000 | [diff] [blame] | 288 | }); |
| 289 | |
| 290 | ASSERT_TRUE(r()->Resolve()) << r()->error(); |
| 291 | } |
| 292 | |
| 293 | TEST_F(ResolverStorageClassValidationTest, UniformBufferVector) { |
| 294 | // var<uniform> g : vec4<f32>; |
| 295 | Global(Source{{56, 78}}, "g", ty.vec4<f32>(), ast::StorageClass::kUniform, |
Ben Clayton | a996ffb | 2022-02-02 23:07:11 +0000 | [diff] [blame] | 296 | ast::AttributeList{ |
| 297 | create<ast::BindingAttribute>(0), |
| 298 | create<ast::GroupAttribute>(0), |
James Price | 7395e29 | 2022-01-19 15:55:56 +0000 | [diff] [blame] | 299 | }); |
| 300 | |
| 301 | ASSERT_TRUE(r()->Resolve()) << r()->error(); |
Ben Clayton | 4270834 | 2021-04-21 17:55:12 +0000 | [diff] [blame] | 302 | } |
| 303 | |
| 304 | TEST_F(ResolverStorageClassValidationTest, UniformBufferArray) { |
James Price | 533a1ba | 2022-01-20 22:11:07 +0000 | [diff] [blame] | 305 | // struct S { |
| 306 | // @size(16) f : f32; |
| 307 | // } |
Ben Clayton | 93e8f52 | 2021-06-04 20:41:47 +0000 | [diff] [blame] | 308 | // var<uniform> g : array<S, 3>; |
James Price | 533a1ba | 2022-01-20 22:11:07 +0000 | [diff] [blame] | 309 | auto* s = Structure("S", {Member("a", ty.f32(), {MemberSize(16)})}); |
Ben Clayton | 8758f10 | 2021-06-09 14:32:14 +0000 | [diff] [blame] | 310 | auto* a = ty.array(ty.Of(s), 3); |
Ben Clayton | 93e8f52 | 2021-06-04 20:41:47 +0000 | [diff] [blame] | 311 | Global(Source{{56, 78}}, "g", a, ast::StorageClass::kUniform, |
Ben Clayton | a996ffb | 2022-02-02 23:07:11 +0000 | [diff] [blame] | 312 | ast::AttributeList{ |
| 313 | create<ast::BindingAttribute>(0), |
| 314 | create<ast::GroupAttribute>(0), |
Ben Clayton | 3f968e7 | 2021-05-10 19:16:46 +0000 | [diff] [blame] | 315 | }); |
Ben Clayton | 4270834 | 2021-04-21 17:55:12 +0000 | [diff] [blame] | 316 | |
James Price | 7395e29 | 2022-01-19 15:55:56 +0000 | [diff] [blame] | 317 | ASSERT_TRUE(r()->Resolve()) << r()->error(); |
Ben Clayton | 4270834 | 2021-04-21 17:55:12 +0000 | [diff] [blame] | 318 | } |
| 319 | |
| 320 | TEST_F(ResolverStorageClassValidationTest, UniformBufferBoolAlias) { |
| 321 | // type a = bool; |
Ben Clayton | 93e8f52 | 2021-06-04 20:41:47 +0000 | [diff] [blame] | 322 | // var<uniform> g : a; |
Ben Clayton | 6e7d24d | 2021-06-09 14:32:14 +0000 | [diff] [blame] | 323 | auto* a = Alias("a", ty.bool_()); |
Ben Clayton | 8758f10 | 2021-06-09 14:32:14 +0000 | [diff] [blame] | 324 | Global(Source{{56, 78}}, "g", ty.Of(a), ast::StorageClass::kUniform, |
Ben Clayton | a996ffb | 2022-02-02 23:07:11 +0000 | [diff] [blame] | 325 | ast::AttributeList{ |
| 326 | create<ast::BindingAttribute>(0), |
| 327 | create<ast::GroupAttribute>(0), |
Ben Clayton | 3f968e7 | 2021-05-10 19:16:46 +0000 | [diff] [blame] | 328 | }); |
Ben Clayton | 4270834 | 2021-04-21 17:55:12 +0000 | [diff] [blame] | 329 | |
| 330 | ASSERT_FALSE(r()->Resolve()); |
| 331 | |
| 332 | EXPECT_EQ( |
| 333 | r()->error(), |
Ben Clayton | a9156ff | 2021-11-05 16:51:38 +0000 | [diff] [blame] | 334 | R"(56:78 error: Type 'bool' cannot be used in storage class 'uniform' as it is non-host-shareable |
| 335 | 56:78 note: while instantiating variable g)"); |
Ben Clayton | 4270834 | 2021-04-21 17:55:12 +0000 | [diff] [blame] | 336 | } |
| 337 | |
Ben Clayton | 4270834 | 2021-04-21 17:55:12 +0000 | [diff] [blame] | 338 | TEST_F(ResolverStorageClassValidationTest, UniformBufferNoError_Basic) { |
| 339 | // [[block]] struct S { x : i32 }; |
| 340 | // var<uniform> g : S; |
Ben Clayton | ba6ab5e | 2021-05-07 14:49:34 +0000 | [diff] [blame] | 341 | auto* s = Structure("S", {Member(Source{{12, 34}}, "x", ty.i32())}, |
Ben Clayton | a996ffb | 2022-02-02 23:07:11 +0000 | [diff] [blame] | 342 | {create<ast::StructBlockAttribute>()}); |
Ben Clayton | 8758f10 | 2021-06-09 14:32:14 +0000 | [diff] [blame] | 343 | Global(Source{{56, 78}}, "g", ty.Of(s), ast::StorageClass::kUniform, |
Ben Clayton | a996ffb | 2022-02-02 23:07:11 +0000 | [diff] [blame] | 344 | ast::AttributeList{ |
| 345 | create<ast::BindingAttribute>(0), |
| 346 | create<ast::GroupAttribute>(0), |
Ben Clayton | 3f968e7 | 2021-05-10 19:16:46 +0000 | [diff] [blame] | 347 | }); |
Ben Clayton | 4270834 | 2021-04-21 17:55:12 +0000 | [diff] [blame] | 348 | |
Antonio Maiorano | 60dae24 | 2021-07-15 19:09:25 +0000 | [diff] [blame] | 349 | ASSERT_TRUE(r()->Resolve()) << r()->error(); |
Ben Clayton | 4270834 | 2021-04-21 17:55:12 +0000 | [diff] [blame] | 350 | } |
| 351 | |
| 352 | TEST_F(ResolverStorageClassValidationTest, UniformBufferNoError_Aliases) { |
| 353 | // [[block]] struct S { x : i32 }; |
| 354 | // type a1 = S; |
| 355 | // var<uniform> g : a1; |
Ben Clayton | ba6ab5e | 2021-05-07 14:49:34 +0000 | [diff] [blame] | 356 | auto* s = Structure("S", {Member(Source{{12, 34}}, "x", ty.i32())}, |
Ben Clayton | a996ffb | 2022-02-02 23:07:11 +0000 | [diff] [blame] | 357 | {create<ast::StructBlockAttribute>()}); |
Ben Clayton | 8758f10 | 2021-06-09 14:32:14 +0000 | [diff] [blame] | 358 | auto* a1 = Alias("a1", ty.Of(s)); |
| 359 | Global(Source{{56, 78}}, "g", ty.Of(a1), ast::StorageClass::kUniform, |
Ben Clayton | a996ffb | 2022-02-02 23:07:11 +0000 | [diff] [blame] | 360 | ast::AttributeList{ |
| 361 | create<ast::BindingAttribute>(0), |
| 362 | create<ast::GroupAttribute>(0), |
Ben Clayton | 3f968e7 | 2021-05-10 19:16:46 +0000 | [diff] [blame] | 363 | }); |
Ben Clayton | 4270834 | 2021-04-21 17:55:12 +0000 | [diff] [blame] | 364 | |
Antonio Maiorano | 60dae24 | 2021-07-15 19:09:25 +0000 | [diff] [blame] | 365 | ASSERT_TRUE(r()->Resolve()) << r()->error(); |
Ben Clayton | 4270834 | 2021-04-21 17:55:12 +0000 | [diff] [blame] | 366 | } |
| 367 | |
Ben Clayton | 1773535 | 2021-04-16 09:26:14 +0000 | [diff] [blame] | 368 | } // namespace |
| 369 | } // namespace resolver |
| 370 | } // namespace tint |