tint: Move resolver/type_alias to type/short_name
'Short-name' is way less overloaded than 'alias' and 'builtin'.
The package move allows transforms to use these enums.
Change-Id: I61c6b3f7deee8e835990a948cd5427c07034fa5e
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/113440
Kokoro: Kokoro <noreply+kokoro@google.com>
Commit-Queue: Ben Clayton <bclayton@google.com>
Reviewed-by: Dan Sinclair <dsinclair@chromium.org>
diff --git a/src/tint/BUILD.gn b/src/tint/BUILD.gn
index c1aeef0..01f36e2 100644
--- a/src/tint/BUILD.gn
+++ b/src/tint/BUILD.gn
@@ -408,8 +408,6 @@
"resolver/resolver.h",
"resolver/sem_helper.cc",
"resolver/sem_helper.h",
- "resolver/type_alias.cc",
- "resolver/type_alias.h",
"resolver/uniformity.cc",
"resolver/uniformity.h",
"resolver/validator.cc",
@@ -574,6 +572,7 @@
"type/reference.h",
"type/sampled_texture.h",
"type/sampler.h",
+ "type/short_name.h",
"type/storage_texture.h",
"type/texture.h",
"type/type.h",
@@ -737,6 +736,8 @@
"type/sampled_texture.h",
"type/sampler.cc",
"type/sampler.h",
+ "type/short_name.cc",
+ "type/short_name.h",
"type/storage_texture.cc",
"type/storage_texture.h",
"type/texture.cc",
@@ -1226,6 +1227,7 @@
"type/reference_test.cc",
"type/sampled_texture_test.cc",
"type/sampler_test.cc",
+ "type/short_name_test.cc",
"type/storage_texture_test.cc",
"type/texture_test.cc",
"type/type_manager_test.cc",
diff --git a/src/tint/CMakeLists.txt b/src/tint/CMakeLists.txt
index 827b055..d691552 100644
--- a/src/tint/CMakeLists.txt
+++ b/src/tint/CMakeLists.txt
@@ -552,9 +552,9 @@
tint_generated(ast/interpolate_attribute BENCH TEST)
tint_generated(ast/texel_format BENCH TEST)
tint_generated(resolver/init_conv_intrinsic)
-tint_generated(resolver/type_alias BENCH TEST)
tint_generated(sem/builtin_type)
tint_generated(sem/parameter_usage)
+tint_generated(type/short_name BENCH TEST)
if(UNIX)
list(APPEND TINT_LIB_SRCS diagnostic/printer_linux.cc)
diff --git a/src/tint/intrinsics.def b/src/tint/intrinsics.def
index 52a30ea..8a58c70 100644
--- a/src/tint/intrinsics.def
+++ b/src/tint/intrinsics.def
@@ -112,7 +112,7 @@
}
// https://www.w3.org/TR/WGSL/#vector-types
-enum type_alias {
+enum short_name {
vec2f
vec2h
vec2i
diff --git a/src/tint/resolver/dependency_graph.cc b/src/tint/resolver/dependency_graph.cc
index 29f88a7..9d2e0da 100644
--- a/src/tint/resolver/dependency_graph.cc
+++ b/src/tint/resolver/dependency_graph.cc
@@ -66,10 +66,10 @@
#include "src/tint/ast/void.h"
#include "src/tint/ast/while_statement.h"
#include "src/tint/ast/workgroup_attribute.h"
-#include "src/tint/resolver/type_alias.h"
#include "src/tint/scope_stack.h"
#include "src/tint/sem/builtin.h"
#include "src/tint/symbol_table.h"
+#include "src/tint/type/short_name.h"
#include "src/tint/utils/block_allocator.h"
#include "src/tint/utils/defer.h"
#include "src/tint/utils/map.h"
@@ -486,7 +486,7 @@
bool IsBuiltin(Symbol name) const {
auto s = symbols_.NameFor(name);
if (sem::ParseBuiltinType(s) != sem::BuiltinType::kNone ||
- ParseTypeAlias(s) != TypeAlias::kUndefined) {
+ type::ParseShortName(s) != type::ShortName::kUndefined) {
return true;
}
return false;
diff --git a/src/tint/resolver/resolver.cc b/src/tint/resolver/resolver.cc
index 31e7abf..df7131c 100644
--- a/src/tint/resolver/resolver.cc
+++ b/src/tint/resolver/resolver.cc
@@ -51,7 +51,6 @@
#include "src/tint/ast/vector.h"
#include "src/tint/ast/while_statement.h"
#include "src/tint/ast/workgroup_attribute.h"
-#include "src/tint/resolver/type_alias.h"
#include "src/tint/resolver/uniformity.h"
#include "src/tint/sem/array.h"
#include "src/tint/sem/atomic.h"
@@ -81,6 +80,7 @@
#include "src/tint/type/reference.h"
#include "src/tint/type/sampled_texture.h"
#include "src/tint/type/sampler.h"
+#include "src/tint/type/short_name.h"
#include "src/tint/type/storage_texture.h"
#include "src/tint/utils/defer.h"
#include "src/tint/utils/math.h"
@@ -2349,32 +2349,32 @@
type::Type* Resolver::BuiltinTypeAlias(Symbol sym) const {
auto name = builder_->Symbols().NameFor(sym);
auto& b = *builder_;
- switch (ParseTypeAlias(name)) {
- case TypeAlias::kVec2F:
+ switch (type::ParseShortName(name)) {
+ case type::ShortName::kVec2F:
return b.create<sem::Vector>(b.create<type::F32>(), 2u);
- case TypeAlias::kVec3F:
+ case type::ShortName::kVec3F:
return b.create<sem::Vector>(b.create<type::F32>(), 3u);
- case TypeAlias::kVec4F:
+ case type::ShortName::kVec4F:
return b.create<sem::Vector>(b.create<type::F32>(), 4u);
- case TypeAlias::kVec2H:
+ case type::ShortName::kVec2H:
return b.create<sem::Vector>(b.create<type::F16>(), 2u);
- case TypeAlias::kVec3H:
+ case type::ShortName::kVec3H:
return b.create<sem::Vector>(b.create<type::F16>(), 3u);
- case TypeAlias::kVec4H:
+ case type::ShortName::kVec4H:
return b.create<sem::Vector>(b.create<type::F16>(), 4u);
- case TypeAlias::kVec2I:
+ case type::ShortName::kVec2I:
return b.create<sem::Vector>(b.create<type::I32>(), 2u);
- case TypeAlias::kVec3I:
+ case type::ShortName::kVec3I:
return b.create<sem::Vector>(b.create<type::I32>(), 3u);
- case TypeAlias::kVec4I:
+ case type::ShortName::kVec4I:
return b.create<sem::Vector>(b.create<type::I32>(), 4u);
- case TypeAlias::kVec2U:
+ case type::ShortName::kVec2U:
return b.create<sem::Vector>(b.create<type::U32>(), 2u);
- case TypeAlias::kVec3U:
+ case type::ShortName::kVec3U:
return b.create<sem::Vector>(b.create<type::U32>(), 3u);
- case TypeAlias::kVec4U:
+ case type::ShortName::kVec4U:
return b.create<sem::Vector>(b.create<type::U32>(), 4u);
- case TypeAlias::kUndefined:
+ case type::ShortName::kUndefined:
break;
}
return nullptr;
diff --git a/src/tint/resolver/type_alias.cc b/src/tint/resolver/type_alias.cc
deleted file mode 100644
index 124fc8c..0000000
--- a/src/tint/resolver/type_alias.cc
+++ /dev/null
@@ -1,102 +0,0 @@
-// Copyright 2022 The Tint Authors.
-//
-// Licensed under the Apache License, Version 2.0 (the "License");
-// you may not use this file except in compliance with the License.
-// You may obtain a copy of the License at
-//
-// http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
-////////////////////////////////////////////////////////////////////////////////
-// File generated by tools/src/cmd/gen
-// using the template:
-// src/tint/resolver/type_alias.cc.tmpl
-//
-// Do not modify this file directly
-////////////////////////////////////////////////////////////////////////////////
-
-#include "src/tint/resolver/type_alias.h"
-
-namespace tint::resolver {
-
-/// ParseTypeAlias parses a TypeAlias from a string.
-/// @param str the string to parse
-/// @returns the parsed enum, or TypeAlias::kUndefined if the string could not be parsed.
-TypeAlias ParseTypeAlias(std::string_view str) {
- if (str == "vec2f") {
- return TypeAlias::kVec2F;
- }
- if (str == "vec2h") {
- return TypeAlias::kVec2H;
- }
- if (str == "vec2i") {
- return TypeAlias::kVec2I;
- }
- if (str == "vec2u") {
- return TypeAlias::kVec2U;
- }
- if (str == "vec3f") {
- return TypeAlias::kVec3F;
- }
- if (str == "vec3h") {
- return TypeAlias::kVec3H;
- }
- if (str == "vec3i") {
- return TypeAlias::kVec3I;
- }
- if (str == "vec3u") {
- return TypeAlias::kVec3U;
- }
- if (str == "vec4f") {
- return TypeAlias::kVec4F;
- }
- if (str == "vec4h") {
- return TypeAlias::kVec4H;
- }
- if (str == "vec4i") {
- return TypeAlias::kVec4I;
- }
- if (str == "vec4u") {
- return TypeAlias::kVec4U;
- }
- return TypeAlias::kUndefined;
-}
-
-std::ostream& operator<<(std::ostream& out, TypeAlias value) {
- switch (value) {
- case TypeAlias::kUndefined:
- return out << "undefined";
- case TypeAlias::kVec2F:
- return out << "vec2f";
- case TypeAlias::kVec2H:
- return out << "vec2h";
- case TypeAlias::kVec2I:
- return out << "vec2i";
- case TypeAlias::kVec2U:
- return out << "vec2u";
- case TypeAlias::kVec3F:
- return out << "vec3f";
- case TypeAlias::kVec3H:
- return out << "vec3h";
- case TypeAlias::kVec3I:
- return out << "vec3i";
- case TypeAlias::kVec3U:
- return out << "vec3u";
- case TypeAlias::kVec4F:
- return out << "vec4f";
- case TypeAlias::kVec4H:
- return out << "vec4h";
- case TypeAlias::kVec4I:
- return out << "vec4i";
- case TypeAlias::kVec4U:
- return out << "vec4u";
- }
- return out << "<unknown>";
-}
-
-} // namespace tint::resolver
diff --git a/src/tint/resolver/type_alias_test.cc b/src/tint/resolver/type_alias_test.cc
deleted file mode 100644
index c52d44c..0000000
--- a/src/tint/resolver/type_alias_test.cc
+++ /dev/null
@@ -1,97 +0,0 @@
-// Copyright 2022 The Tint Authors.
-//
-// Licensed under the Apache License, Version 2.0 (the "License");
-// you may not use this file except in compliance with the License.
-// You may obtain a copy of the License at
-//
-// http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
-////////////////////////////////////////////////////////////////////////////////
-// File generated by tools/src/cmd/gen
-// using the template:
-// src/tint/resolver/type_alias_test.cc.tmpl
-//
-// Do not modify this file directly
-////////////////////////////////////////////////////////////////////////////////
-
-#include "src/tint/resolver/type_alias.h"
-
-#include <string>
-
-#include "gtest/gtest.h"
-
-#include "src/tint/utils/string.h"
-
-namespace tint::resolver {
-namespace {
-
-namespace parse_print_tests {
-
-struct Case {
- const char* string;
- TypeAlias value;
-};
-
-inline std::ostream& operator<<(std::ostream& out, Case c) {
- return out << "'" << std::string(c.string) << "'";
-}
-
-static constexpr Case kValidCases[] = {
- {"vec2f", TypeAlias::kVec2F}, {"vec2h", TypeAlias::kVec2H}, {"vec2i", TypeAlias::kVec2I},
- {"vec2u", TypeAlias::kVec2U}, {"vec3f", TypeAlias::kVec3F}, {"vec3h", TypeAlias::kVec3H},
- {"vec3i", TypeAlias::kVec3I}, {"vec3u", TypeAlias::kVec3U}, {"vec4f", TypeAlias::kVec4F},
- {"vec4h", TypeAlias::kVec4H}, {"vec4i", TypeAlias::kVec4I}, {"vec4u", TypeAlias::kVec4U},
-};
-
-static constexpr Case kInvalidCases[] = {
- {"veccf", TypeAlias::kUndefined}, {"32", TypeAlias::kUndefined},
- {"vVc2f", TypeAlias::kUndefined}, {"vec21", TypeAlias::kUndefined},
- {"qqeJh", TypeAlias::kUndefined}, {"vecll7h", TypeAlias::kUndefined},
- {"veqH2pp", TypeAlias::kUndefined}, {"vi", TypeAlias::kUndefined},
- {"Gebi", TypeAlias::kUndefined}, {"vevi2u", TypeAlias::kUndefined},
- {"ve8WWu", TypeAlias::kUndefined}, {"Mxxc2", TypeAlias::kUndefined},
- {"vgg3f", TypeAlias::kUndefined}, {"V3X", TypeAlias::kUndefined},
- {"vec33", TypeAlias::kUndefined}, {"vec3E", TypeAlias::kUndefined},
- {"TTeP3h", TypeAlias::kUndefined}, {"vxxcdd", TypeAlias::kUndefined},
- {"v44c3i", TypeAlias::kUndefined}, {"veVVSSi", TypeAlias::kUndefined},
- {"22RRi", TypeAlias::kUndefined}, {"vF3u", TypeAlias::kUndefined},
- {"vecu", TypeAlias::kUndefined}, {"ROOHVu", TypeAlias::kUndefined},
- {"ecyf", TypeAlias::kUndefined}, {"n77rrlcGf", TypeAlias::kUndefined},
- {"vec440", TypeAlias::kUndefined}, {"ooh", TypeAlias::kUndefined},
- {"vezz", TypeAlias::kUndefined}, {"1ipp4h", TypeAlias::kUndefined},
- {"XXec4i", TypeAlias::kUndefined}, {"ve9IInn5i", TypeAlias::kUndefined},
- {"HHreSSaYi", TypeAlias::kUndefined}, {"kk4", TypeAlias::kUndefined},
- {"jgRR", TypeAlias::kUndefined}, {"veb", TypeAlias::kUndefined},
-};
-
-using TypeAliasParseTest = testing::TestWithParam<Case>;
-
-TEST_P(TypeAliasParseTest, Parse) {
- const char* string = GetParam().string;
- TypeAlias expect = GetParam().value;
- EXPECT_EQ(expect, ParseTypeAlias(string));
-}
-
-INSTANTIATE_TEST_SUITE_P(ValidCases, TypeAliasParseTest, testing::ValuesIn(kValidCases));
-INSTANTIATE_TEST_SUITE_P(InvalidCases, TypeAliasParseTest, testing::ValuesIn(kInvalidCases));
-
-using TypeAliasPrintTest = testing::TestWithParam<Case>;
-
-TEST_P(TypeAliasPrintTest, Print) {
- TypeAlias value = GetParam().value;
- const char* expect = GetParam().string;
- EXPECT_EQ(expect, utils::ToString(value));
-}
-
-INSTANTIATE_TEST_SUITE_P(ValidCases, TypeAliasPrintTest, testing::ValuesIn(kValidCases));
-
-} // namespace parse_print_tests
-
-} // namespace
-} // namespace tint::resolver
diff --git a/src/tint/type/short_name.cc b/src/tint/type/short_name.cc
new file mode 100644
index 0000000..1e84627
--- /dev/null
+++ b/src/tint/type/short_name.cc
@@ -0,0 +1,102 @@
+// Copyright 2022 The Tint Authors.
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+// http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+////////////////////////////////////////////////////////////////////////////////
+// File generated by tools/src/cmd/gen
+// using the template:
+// src/tint/type/short_name.cc.tmpl
+//
+// Do not modify this file directly
+////////////////////////////////////////////////////////////////////////////////
+
+#include "src/tint/type/short_name.h"
+
+namespace tint::type {
+
+/// ParseShortName parses a ShortName from a string.
+/// @param str the string to parse
+/// @returns the parsed enum, or ShortName::kUndefined if the string could not be parsed.
+ShortName ParseShortName(std::string_view str) {
+ if (str == "vec2f") {
+ return ShortName::kVec2F;
+ }
+ if (str == "vec2h") {
+ return ShortName::kVec2H;
+ }
+ if (str == "vec2i") {
+ return ShortName::kVec2I;
+ }
+ if (str == "vec2u") {
+ return ShortName::kVec2U;
+ }
+ if (str == "vec3f") {
+ return ShortName::kVec3F;
+ }
+ if (str == "vec3h") {
+ return ShortName::kVec3H;
+ }
+ if (str == "vec3i") {
+ return ShortName::kVec3I;
+ }
+ if (str == "vec3u") {
+ return ShortName::kVec3U;
+ }
+ if (str == "vec4f") {
+ return ShortName::kVec4F;
+ }
+ if (str == "vec4h") {
+ return ShortName::kVec4H;
+ }
+ if (str == "vec4i") {
+ return ShortName::kVec4I;
+ }
+ if (str == "vec4u") {
+ return ShortName::kVec4U;
+ }
+ return ShortName::kUndefined;
+}
+
+std::ostream& operator<<(std::ostream& out, ShortName value) {
+ switch (value) {
+ case ShortName::kUndefined:
+ return out << "undefined";
+ case ShortName::kVec2F:
+ return out << "vec2f";
+ case ShortName::kVec2H:
+ return out << "vec2h";
+ case ShortName::kVec2I:
+ return out << "vec2i";
+ case ShortName::kVec2U:
+ return out << "vec2u";
+ case ShortName::kVec3F:
+ return out << "vec3f";
+ case ShortName::kVec3H:
+ return out << "vec3h";
+ case ShortName::kVec3I:
+ return out << "vec3i";
+ case ShortName::kVec3U:
+ return out << "vec3u";
+ case ShortName::kVec4F:
+ return out << "vec4f";
+ case ShortName::kVec4H:
+ return out << "vec4h";
+ case ShortName::kVec4I:
+ return out << "vec4i";
+ case ShortName::kVec4U:
+ return out << "vec4u";
+ }
+ return out << "<unknown>";
+}
+
+} // namespace tint::type
diff --git a/src/tint/resolver/type_alias.cc.tmpl b/src/tint/type/short_name.cc.tmpl
similarity index 69%
rename from src/tint/resolver/type_alias.cc.tmpl
rename to src/tint/type/short_name.cc.tmpl
index 3843864..465feb8 100644
--- a/src/tint/resolver/type_alias.cc.tmpl
+++ b/src/tint/type/short_name.cc.tmpl
@@ -1,6 +1,6 @@
{{- /*
--------------------------------------------------------------------------------
-Template file for use with tools/src/cmd/gen to generate type_alias.cc
+Template file for use with tools/src/cmd/gen to generate short_name.cc
To update the generated file, run:
./tools/run gen
@@ -12,14 +12,14 @@
*/ -}}
{{- Import "src/tint/templates/enums.tmpl.inc" -}}
-{{- $enum := (Sem.Enum "type_alias") -}}
+{{- $enum := (Sem.Enum "short_name") -}}
-#include "src/tint/resolver/type_alias.h"
+#include "src/tint/type/short_name.h"
-namespace tint::resolver {
+namespace tint::type {
{{ Eval "ParseEnum" $enum}}
{{ Eval "EnumOStream" $enum}}
-} // namespace tint::resolver
+} // namespace tint::type
diff --git a/src/tint/resolver/type_alias.h b/src/tint/type/short_name.h
similarity index 70%
rename from src/tint/resolver/type_alias.h
rename to src/tint/type/short_name.h
index 40083a3..8b3148f 100644
--- a/src/tint/resolver/type_alias.h
+++ b/src/tint/type/short_name.h
@@ -15,20 +15,20 @@
////////////////////////////////////////////////////////////////////////////////
// File generated by tools/src/cmd/gen
// using the template:
-// src/tint/resolver/type_alias.h.tmpl
+// src/tint/type/short_name.h.tmpl
//
// Do not modify this file directly
////////////////////////////////////////////////////////////////////////////////
-#ifndef SRC_TINT_RESOLVER_TYPE_ALIAS_H_
-#define SRC_TINT_RESOLVER_TYPE_ALIAS_H_
+#ifndef SRC_TINT_TYPE_SHORT_NAME_H_
+#define SRC_TINT_TYPE_SHORT_NAME_H_
#include <ostream>
-namespace tint::resolver {
+namespace tint::type {
/// An enumerator of builtin type aliases.
-enum class TypeAlias {
+enum class ShortName {
kUndefined,
kVec2F,
kVec2H,
@@ -45,20 +45,20 @@
};
/// @param out the std::ostream to write to
-/// @param value the TypeAlias
+/// @param value the ShortName
/// @returns `out` so calls can be chained
-std::ostream& operator<<(std::ostream& out, TypeAlias value);
+std::ostream& operator<<(std::ostream& out, ShortName value);
-/// ParseTypeAlias parses a TypeAlias from a string.
+/// ParseShortName parses a ShortName from a string.
/// @param str the string to parse
-/// @returns the parsed enum, or TypeAlias::kUndefined if the string could not be parsed.
-TypeAlias ParseTypeAlias(std::string_view str);
+/// @returns the parsed enum, or ShortName::kUndefined if the string could not be parsed.
+ShortName ParseShortName(std::string_view str);
-constexpr const char* kTypeAliasStrings[] = {
+constexpr const char* kShortNameStrings[] = {
"vec2f", "vec2h", "vec2i", "vec2u", "vec3f", "vec3h",
"vec3i", "vec3u", "vec4f", "vec4h", "vec4i", "vec4u",
};
-} // namespace tint::resolver
+} // namespace tint::type
-#endif // SRC_TINT_RESOLVER_TYPE_ALIAS_H_
+#endif // SRC_TINT_TYPE_SHORT_NAME_H_
diff --git a/src/tint/resolver/type_alias.h.tmpl b/src/tint/type/short_name.h.tmpl
similarity index 63%
rename from src/tint/resolver/type_alias.h.tmpl
rename to src/tint/type/short_name.h.tmpl
index 0a9b2d4..300cbfc 100644
--- a/src/tint/resolver/type_alias.h.tmpl
+++ b/src/tint/type/short_name.h.tmpl
@@ -1,6 +1,6 @@
{{- /*
--------------------------------------------------------------------------------
-Template file for use with tools/src/cmd/gen to generate type_alias.h
+Template file for use with tools/src/cmd/gen to generate short_name.h
To update the generated file, run:
./tools/run gen
@@ -12,18 +12,18 @@
*/ -}}
{{- Import "src/tint/templates/enums.tmpl.inc" -}}
-{{- $enum := (Sem.Enum "type_alias") -}}
+{{- $enum := (Sem.Enum "short_name") -}}
-#ifndef SRC_TINT_RESOLVER_TYPE_ALIAS_H_
-#define SRC_TINT_RESOLVER_TYPE_ALIAS_H_
+#ifndef SRC_TINT_TYPE_SHORT_NAME_H_
+#define SRC_TINT_TYPE_SHORT_NAME_H_
#include <ostream>
-namespace tint::resolver {
+namespace tint::type {
/// An enumerator of builtin type aliases.
{{ Eval "DeclareEnum" $enum}}
-} // namespace tint::resolver
+} // namespace tint::type
-#endif // SRC_TINT_RESOLVER_TYPE_ALIAS_H_
+#endif // SRC_TINT_TYPE_SHORT_NAME_H_
diff --git a/src/tint/resolver/type_alias_bench.cc b/src/tint/type/short_name_bench.cc
similarity index 87%
rename from src/tint/resolver/type_alias_bench.cc
rename to src/tint/type/short_name_bench.cc
index 2510352..93e3551 100644
--- a/src/tint/resolver/type_alias_bench.cc
+++ b/src/tint/type/short_name_bench.cc
@@ -15,21 +15,21 @@
////////////////////////////////////////////////////////////////////////////////
// File generated by tools/src/cmd/gen
// using the template:
-// src/tint/resolver/type_alias_bench.cc.tmpl
+// src/tint/type/short_name_bench.cc.tmpl
//
// Do not modify this file directly
////////////////////////////////////////////////////////////////////////////////
-#include "src/tint/resolver/type_alias.h"
+#include "src/tint/type/short_name.h"
#include <array>
#include "benchmark/benchmark.h"
-namespace tint::resolver {
+namespace tint::type {
namespace {
-void TypeAliasParser(::benchmark::State& state) {
+void ShortNameParser(::benchmark::State& state) {
std::array kStrings{
"veccf", "32", "vVc2f", "vec2f", "vec21", "qqeJf", "vecll7f", "veqH2pp",
"vh", "Gebh", "vec2h", "vevi2h", "ve8WWh", "Mxxc2", "vgg2i", "V2X",
@@ -45,13 +45,13 @@
};
for (auto _ : state) {
for (auto& str : kStrings) {
- auto result = ParseTypeAlias(str);
+ auto result = ParseShortName(str);
benchmark::DoNotOptimize(result);
}
}
}
-BENCHMARK(TypeAliasParser);
+BENCHMARK(ShortNameParser);
} // namespace
-} // namespace tint::resolver
+} // namespace tint::type
diff --git a/src/tint/resolver/type_alias_bench.cc.tmpl b/src/tint/type/short_name_bench.cc.tmpl
similarity index 71%
rename from src/tint/resolver/type_alias_bench.cc.tmpl
rename to src/tint/type/short_name_bench.cc.tmpl
index 3a48226..079e4e1 100644
--- a/src/tint/resolver/type_alias_bench.cc.tmpl
+++ b/src/tint/type/short_name_bench.cc.tmpl
@@ -1,6 +1,6 @@
{{- /*
--------------------------------------------------------------------------------
-Template file for use with tools/src/cmd/gen to generate type_alias_bench.cc
+Template file for use with tools/src/cmd/gen to generate short_name_bench.cc
To update the generated file, run:
./tools/run gen
@@ -12,18 +12,18 @@
*/ -}}
{{- Import "src/tint/templates/enums.tmpl.inc" -}}
-{{- $enum := (Sem.Enum "type_alias") -}}
+{{- $enum := (Sem.Enum "short_name") -}}
-#include "src/tint/resolver/type_alias.h"
+#include "src/tint/type/short_name.h"
#include <array>
#include "benchmark/benchmark.h"
-namespace tint::resolver {
+namespace tint::type {
namespace {
{{ Eval "BenchmarkParseEnum" $enum }}
} // namespace
-} // namespace tint::resolver
+} // namespace tint::type
diff --git a/src/tint/type/short_name_test.cc b/src/tint/type/short_name_test.cc
new file mode 100644
index 0000000..6be0076
--- /dev/null
+++ b/src/tint/type/short_name_test.cc
@@ -0,0 +1,97 @@
+// Copyright 2022 The Tint Authors.
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+// http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+////////////////////////////////////////////////////////////////////////////////
+// File generated by tools/src/cmd/gen
+// using the template:
+// src/tint/type/short_name_test.cc.tmpl
+//
+// Do not modify this file directly
+////////////////////////////////////////////////////////////////////////////////
+
+#include "src/tint/type/short_name.h"
+
+#include <string>
+
+#include "gtest/gtest.h"
+
+#include "src/tint/utils/string.h"
+
+namespace tint::type {
+namespace {
+
+namespace parse_print_tests {
+
+struct Case {
+ const char* string;
+ ShortName value;
+};
+
+inline std::ostream& operator<<(std::ostream& out, Case c) {
+ return out << "'" << std::string(c.string) << "'";
+}
+
+static constexpr Case kValidCases[] = {
+ {"vec2f", ShortName::kVec2F}, {"vec2h", ShortName::kVec2H}, {"vec2i", ShortName::kVec2I},
+ {"vec2u", ShortName::kVec2U}, {"vec3f", ShortName::kVec3F}, {"vec3h", ShortName::kVec3H},
+ {"vec3i", ShortName::kVec3I}, {"vec3u", ShortName::kVec3U}, {"vec4f", ShortName::kVec4F},
+ {"vec4h", ShortName::kVec4H}, {"vec4i", ShortName::kVec4I}, {"vec4u", ShortName::kVec4U},
+};
+
+static constexpr Case kInvalidCases[] = {
+ {"veccf", ShortName::kUndefined}, {"32", ShortName::kUndefined},
+ {"vVc2f", ShortName::kUndefined}, {"vec21", ShortName::kUndefined},
+ {"qqeJh", ShortName::kUndefined}, {"vecll7h", ShortName::kUndefined},
+ {"veqH2pp", ShortName::kUndefined}, {"vi", ShortName::kUndefined},
+ {"Gebi", ShortName::kUndefined}, {"vevi2u", ShortName::kUndefined},
+ {"ve8WWu", ShortName::kUndefined}, {"Mxxc2", ShortName::kUndefined},
+ {"vgg3f", ShortName::kUndefined}, {"V3X", ShortName::kUndefined},
+ {"vec33", ShortName::kUndefined}, {"vec3E", ShortName::kUndefined},
+ {"TTeP3h", ShortName::kUndefined}, {"vxxcdd", ShortName::kUndefined},
+ {"v44c3i", ShortName::kUndefined}, {"veVVSSi", ShortName::kUndefined},
+ {"22RRi", ShortName::kUndefined}, {"vF3u", ShortName::kUndefined},
+ {"vecu", ShortName::kUndefined}, {"ROOHVu", ShortName::kUndefined},
+ {"ecyf", ShortName::kUndefined}, {"n77rrlcGf", ShortName::kUndefined},
+ {"vec440", ShortName::kUndefined}, {"ooh", ShortName::kUndefined},
+ {"vezz", ShortName::kUndefined}, {"1ipp4h", ShortName::kUndefined},
+ {"XXec4i", ShortName::kUndefined}, {"ve9IInn5i", ShortName::kUndefined},
+ {"HHreSSaYi", ShortName::kUndefined}, {"kk4", ShortName::kUndefined},
+ {"jgRR", ShortName::kUndefined}, {"veb", ShortName::kUndefined},
+};
+
+using ShortNameParseTest = testing::TestWithParam<Case>;
+
+TEST_P(ShortNameParseTest, Parse) {
+ const char* string = GetParam().string;
+ ShortName expect = GetParam().value;
+ EXPECT_EQ(expect, ParseShortName(string));
+}
+
+INSTANTIATE_TEST_SUITE_P(ValidCases, ShortNameParseTest, testing::ValuesIn(kValidCases));
+INSTANTIATE_TEST_SUITE_P(InvalidCases, ShortNameParseTest, testing::ValuesIn(kInvalidCases));
+
+using ShortNamePrintTest = testing::TestWithParam<Case>;
+
+TEST_P(ShortNamePrintTest, Print) {
+ ShortName value = GetParam().value;
+ const char* expect = GetParam().string;
+ EXPECT_EQ(expect, utils::ToString(value));
+}
+
+INSTANTIATE_TEST_SUITE_P(ValidCases, ShortNamePrintTest, testing::ValuesIn(kValidCases));
+
+} // namespace parse_print_tests
+
+} // namespace
+} // namespace tint::type
diff --git a/src/tint/resolver/type_alias_test.cc.tmpl b/src/tint/type/short_name_test.cc.tmpl
similarity index 72%
rename from src/tint/resolver/type_alias_test.cc.tmpl
rename to src/tint/type/short_name_test.cc.tmpl
index 2f37251..4e27e42 100644
--- a/src/tint/resolver/type_alias_test.cc.tmpl
+++ b/src/tint/type/short_name_test.cc.tmpl
@@ -1,6 +1,6 @@
{{- /*
--------------------------------------------------------------------------------
-Template file for use with tools/src/cmd/gen to generate type_alias_test.cc
+Template file for use with tools/src/cmd/gen to generate short_name_test.cc
To update the generated file, run:
./tools/run gen
@@ -12,9 +12,9 @@
*/ -}}
{{- Import "src/tint/templates/enums.tmpl.inc" -}}
-{{- $enum := (Sem.Enum "type_alias") -}}
+{{- $enum := (Sem.Enum "short_name") -}}
-#include "src/tint/resolver/type_alias.h"
+#include "src/tint/type/short_name.h"
#include <string>
@@ -22,10 +22,10 @@
#include "src/tint/utils/string.h"
-namespace tint::resolver {
+namespace tint::type {
namespace {
{{ Eval "TestParsePrintEnum" $enum}}
} // namespace
-} // namespace tint::resolver
+} // namespace tint::type
diff --git a/test/tint/shadowing/alias/builtin/const.wgsl.expected.dxc.hlsl b/test/tint/shadowing/alias/builtin/const.wgsl.expected.dxc.hlsl
deleted file mode 100644
index 6316714..0000000
--- a/test/tint/shadowing/alias/builtin/const.wgsl.expected.dxc.hlsl
+++ /dev/null
@@ -1,9 +0,0 @@
-[numthreads(1, 1, 1)]
-void unused_entry_point() {
- return;
-}
-
-void f() {
- {
- }
-}
diff --git a/test/tint/shadowing/alias/builtin/const.wgsl.expected.fxc.hlsl b/test/tint/shadowing/alias/builtin/const.wgsl.expected.fxc.hlsl
deleted file mode 100644
index 6316714..0000000
--- a/test/tint/shadowing/alias/builtin/const.wgsl.expected.fxc.hlsl
+++ /dev/null
@@ -1,9 +0,0 @@
-[numthreads(1, 1, 1)]
-void unused_entry_point() {
- return;
-}
-
-void f() {
- {
- }
-}
diff --git a/test/tint/shadowing/alias/builtin/const.wgsl.expected.glsl b/test/tint/shadowing/alias/builtin/const.wgsl.expected.glsl
deleted file mode 100644
index 0da98e3..0000000
--- a/test/tint/shadowing/alias/builtin/const.wgsl.expected.glsl
+++ /dev/null
@@ -1,11 +0,0 @@
-#version 310 es
-
-layout(local_size_x = 1, local_size_y = 1, local_size_z = 1) in;
-void unused_entry_point() {
- return;
-}
-void f() {
- {
- }
-}
-
diff --git a/test/tint/shadowing/alias/builtin/const.wgsl.expected.msl b/test/tint/shadowing/alias/builtin/const.wgsl.expected.msl
deleted file mode 100644
index ee183f0..0000000
--- a/test/tint/shadowing/alias/builtin/const.wgsl.expected.msl
+++ /dev/null
@@ -1,8 +0,0 @@
-#include <metal_stdlib>
-
-using namespace metal;
-void f() {
- {
- }
-}
-
diff --git a/test/tint/shadowing/alias/builtin/const.wgsl.expected.spvasm b/test/tint/shadowing/alias/builtin/const.wgsl.expected.spvasm
deleted file mode 100644
index 1c4d684..0000000
--- a/test/tint/shadowing/alias/builtin/const.wgsl.expected.spvasm
+++ /dev/null
@@ -1,21 +0,0 @@
-; SPIR-V
-; Version: 1.3
-; Generator: Google Tint Compiler; 0
-; Bound: 7
-; Schema: 0
- OpCapability Shader
- OpMemoryModel Logical GLSL450
- OpEntryPoint GLCompute %unused_entry_point "unused_entry_point"
- OpExecutionMode %unused_entry_point LocalSize 1 1 1
- OpName %unused_entry_point "unused_entry_point"
- OpName %f "f"
- %void = OpTypeVoid
- %1 = OpTypeFunction %void
-%unused_entry_point = OpFunction %void None %1
- %4 = OpLabel
- OpReturn
- OpFunctionEnd
- %f = OpFunction %void None %1
- %6 = OpLabel
- OpReturn
- OpFunctionEnd
diff --git a/test/tint/shadowing/alias/user/const.wgsl b/test/tint/shadowing/alias/const.wgsl
similarity index 100%
rename from test/tint/shadowing/alias/user/const.wgsl
rename to test/tint/shadowing/alias/const.wgsl
diff --git a/test/tint/shadowing/alias/user/const.wgsl.expected.dxc.hlsl b/test/tint/shadowing/alias/const.wgsl.expected.dxc.hlsl
similarity index 100%
rename from test/tint/shadowing/alias/user/const.wgsl.expected.dxc.hlsl
rename to test/tint/shadowing/alias/const.wgsl.expected.dxc.hlsl
diff --git a/test/tint/shadowing/alias/user/const.wgsl.expected.fxc.hlsl b/test/tint/shadowing/alias/const.wgsl.expected.fxc.hlsl
similarity index 100%
rename from test/tint/shadowing/alias/user/const.wgsl.expected.fxc.hlsl
rename to test/tint/shadowing/alias/const.wgsl.expected.fxc.hlsl
diff --git a/test/tint/shadowing/alias/user/const.wgsl.expected.glsl b/test/tint/shadowing/alias/const.wgsl.expected.glsl
similarity index 100%
rename from test/tint/shadowing/alias/user/const.wgsl.expected.glsl
rename to test/tint/shadowing/alias/const.wgsl.expected.glsl
diff --git a/test/tint/shadowing/alias/user/const.wgsl.expected.msl b/test/tint/shadowing/alias/const.wgsl.expected.msl
similarity index 100%
rename from test/tint/shadowing/alias/user/const.wgsl.expected.msl
rename to test/tint/shadowing/alias/const.wgsl.expected.msl
diff --git a/test/tint/shadowing/alias/user/const.wgsl.expected.spvasm b/test/tint/shadowing/alias/const.wgsl.expected.spvasm
similarity index 100%
rename from test/tint/shadowing/alias/user/const.wgsl.expected.spvasm
rename to test/tint/shadowing/alias/const.wgsl.expected.spvasm
diff --git a/test/tint/shadowing/alias/user/const.wgsl.expected.wgsl b/test/tint/shadowing/alias/const.wgsl.expected.wgsl
similarity index 100%
rename from test/tint/shadowing/alias/user/const.wgsl.expected.wgsl
rename to test/tint/shadowing/alias/const.wgsl.expected.wgsl
diff --git a/test/tint/shadowing/alias/user/let.wgsl b/test/tint/shadowing/alias/let.wgsl
similarity index 100%
rename from test/tint/shadowing/alias/user/let.wgsl
rename to test/tint/shadowing/alias/let.wgsl
diff --git a/test/tint/shadowing/alias/user/let.wgsl.expected.dxc.hlsl b/test/tint/shadowing/alias/let.wgsl.expected.dxc.hlsl
similarity index 100%
rename from test/tint/shadowing/alias/user/let.wgsl.expected.dxc.hlsl
rename to test/tint/shadowing/alias/let.wgsl.expected.dxc.hlsl
diff --git a/test/tint/shadowing/alias/user/let.wgsl.expected.fxc.hlsl b/test/tint/shadowing/alias/let.wgsl.expected.fxc.hlsl
similarity index 100%
rename from test/tint/shadowing/alias/user/let.wgsl.expected.fxc.hlsl
rename to test/tint/shadowing/alias/let.wgsl.expected.fxc.hlsl
diff --git a/test/tint/shadowing/alias/user/let.wgsl.expected.glsl b/test/tint/shadowing/alias/let.wgsl.expected.glsl
similarity index 100%
rename from test/tint/shadowing/alias/user/let.wgsl.expected.glsl
rename to test/tint/shadowing/alias/let.wgsl.expected.glsl
diff --git a/test/tint/shadowing/alias/user/let.wgsl.expected.msl b/test/tint/shadowing/alias/let.wgsl.expected.msl
similarity index 100%
rename from test/tint/shadowing/alias/user/let.wgsl.expected.msl
rename to test/tint/shadowing/alias/let.wgsl.expected.msl
diff --git a/test/tint/shadowing/alias/user/let.wgsl.expected.spvasm b/test/tint/shadowing/alias/let.wgsl.expected.spvasm
similarity index 100%
rename from test/tint/shadowing/alias/user/let.wgsl.expected.spvasm
rename to test/tint/shadowing/alias/let.wgsl.expected.spvasm
diff --git a/test/tint/shadowing/alias/user/let.wgsl.expected.wgsl b/test/tint/shadowing/alias/let.wgsl.expected.wgsl
similarity index 100%
rename from test/tint/shadowing/alias/user/let.wgsl.expected.wgsl
rename to test/tint/shadowing/alias/let.wgsl.expected.wgsl
diff --git a/test/tint/shadowing/alias/user/param.wgsl b/test/tint/shadowing/alias/param.wgsl
similarity index 100%
rename from test/tint/shadowing/alias/user/param.wgsl
rename to test/tint/shadowing/alias/param.wgsl
diff --git a/test/tint/shadowing/alias/user/param.wgsl.expected.dxc.hlsl b/test/tint/shadowing/alias/param.wgsl.expected.dxc.hlsl
similarity index 100%
rename from test/tint/shadowing/alias/user/param.wgsl.expected.dxc.hlsl
rename to test/tint/shadowing/alias/param.wgsl.expected.dxc.hlsl
diff --git a/test/tint/shadowing/alias/user/param.wgsl.expected.fxc.hlsl b/test/tint/shadowing/alias/param.wgsl.expected.fxc.hlsl
similarity index 100%
rename from test/tint/shadowing/alias/user/param.wgsl.expected.fxc.hlsl
rename to test/tint/shadowing/alias/param.wgsl.expected.fxc.hlsl
diff --git a/test/tint/shadowing/alias/user/param.wgsl.expected.glsl b/test/tint/shadowing/alias/param.wgsl.expected.glsl
similarity index 100%
rename from test/tint/shadowing/alias/user/param.wgsl.expected.glsl
rename to test/tint/shadowing/alias/param.wgsl.expected.glsl
diff --git a/test/tint/shadowing/alias/user/param.wgsl.expected.msl b/test/tint/shadowing/alias/param.wgsl.expected.msl
similarity index 100%
rename from test/tint/shadowing/alias/user/param.wgsl.expected.msl
rename to test/tint/shadowing/alias/param.wgsl.expected.msl
diff --git a/test/tint/shadowing/alias/user/param.wgsl.expected.spvasm b/test/tint/shadowing/alias/param.wgsl.expected.spvasm
similarity index 100%
rename from test/tint/shadowing/alias/user/param.wgsl.expected.spvasm
rename to test/tint/shadowing/alias/param.wgsl.expected.spvasm
diff --git a/test/tint/shadowing/alias/user/param.wgsl.expected.wgsl b/test/tint/shadowing/alias/param.wgsl.expected.wgsl
similarity index 100%
rename from test/tint/shadowing/alias/user/param.wgsl.expected.wgsl
rename to test/tint/shadowing/alias/param.wgsl.expected.wgsl
diff --git a/test/tint/shadowing/alias/user/var.wgsl b/test/tint/shadowing/alias/var.wgsl
similarity index 100%
rename from test/tint/shadowing/alias/user/var.wgsl
rename to test/tint/shadowing/alias/var.wgsl
diff --git a/test/tint/shadowing/alias/user/var.wgsl.expected.dxc.hlsl b/test/tint/shadowing/alias/var.wgsl.expected.dxc.hlsl
similarity index 100%
rename from test/tint/shadowing/alias/user/var.wgsl.expected.dxc.hlsl
rename to test/tint/shadowing/alias/var.wgsl.expected.dxc.hlsl
diff --git a/test/tint/shadowing/alias/user/var.wgsl.expected.fxc.hlsl b/test/tint/shadowing/alias/var.wgsl.expected.fxc.hlsl
similarity index 100%
rename from test/tint/shadowing/alias/user/var.wgsl.expected.fxc.hlsl
rename to test/tint/shadowing/alias/var.wgsl.expected.fxc.hlsl
diff --git a/test/tint/shadowing/alias/user/var.wgsl.expected.glsl b/test/tint/shadowing/alias/var.wgsl.expected.glsl
similarity index 100%
rename from test/tint/shadowing/alias/user/var.wgsl.expected.glsl
rename to test/tint/shadowing/alias/var.wgsl.expected.glsl
diff --git a/test/tint/shadowing/alias/user/var.wgsl.expected.msl b/test/tint/shadowing/alias/var.wgsl.expected.msl
similarity index 100%
rename from test/tint/shadowing/alias/user/var.wgsl.expected.msl
rename to test/tint/shadowing/alias/var.wgsl.expected.msl
diff --git a/test/tint/shadowing/alias/user/var.wgsl.expected.spvasm b/test/tint/shadowing/alias/var.wgsl.expected.spvasm
similarity index 100%
rename from test/tint/shadowing/alias/user/var.wgsl.expected.spvasm
rename to test/tint/shadowing/alias/var.wgsl.expected.spvasm
diff --git a/test/tint/shadowing/alias/user/var.wgsl.expected.wgsl b/test/tint/shadowing/alias/var.wgsl.expected.wgsl
similarity index 100%
rename from test/tint/shadowing/alias/user/var.wgsl.expected.wgsl
rename to test/tint/shadowing/alias/var.wgsl.expected.wgsl
diff --git a/test/tint/shadowing/alias/builtin/const.wgsl b/test/tint/shadowing/short_names/const.wgsl
similarity index 100%
rename from test/tint/shadowing/alias/builtin/const.wgsl
rename to test/tint/shadowing/short_names/const.wgsl
diff --git a/test/tint/shadowing/alias/user/const.wgsl.expected.dxc.hlsl b/test/tint/shadowing/short_names/const.wgsl.expected.dxc.hlsl
similarity index 100%
copy from test/tint/shadowing/alias/user/const.wgsl.expected.dxc.hlsl
copy to test/tint/shadowing/short_names/const.wgsl.expected.dxc.hlsl
diff --git a/test/tint/shadowing/alias/user/const.wgsl.expected.fxc.hlsl b/test/tint/shadowing/short_names/const.wgsl.expected.fxc.hlsl
similarity index 100%
copy from test/tint/shadowing/alias/user/const.wgsl.expected.fxc.hlsl
copy to test/tint/shadowing/short_names/const.wgsl.expected.fxc.hlsl
diff --git a/test/tint/shadowing/alias/user/const.wgsl.expected.glsl b/test/tint/shadowing/short_names/const.wgsl.expected.glsl
similarity index 100%
copy from test/tint/shadowing/alias/user/const.wgsl.expected.glsl
copy to test/tint/shadowing/short_names/const.wgsl.expected.glsl
diff --git a/test/tint/shadowing/alias/user/const.wgsl.expected.msl b/test/tint/shadowing/short_names/const.wgsl.expected.msl
similarity index 100%
copy from test/tint/shadowing/alias/user/const.wgsl.expected.msl
copy to test/tint/shadowing/short_names/const.wgsl.expected.msl
diff --git a/test/tint/shadowing/alias/user/const.wgsl.expected.spvasm b/test/tint/shadowing/short_names/const.wgsl.expected.spvasm
similarity index 100%
copy from test/tint/shadowing/alias/user/const.wgsl.expected.spvasm
copy to test/tint/shadowing/short_names/const.wgsl.expected.spvasm
diff --git a/test/tint/shadowing/alias/builtin/const.wgsl.expected.wgsl b/test/tint/shadowing/short_names/const.wgsl.expected.wgsl
similarity index 100%
rename from test/tint/shadowing/alias/builtin/const.wgsl.expected.wgsl
rename to test/tint/shadowing/short_names/const.wgsl.expected.wgsl
diff --git a/test/tint/shadowing/alias/builtin/let.wgsl b/test/tint/shadowing/short_names/let.wgsl
similarity index 100%
rename from test/tint/shadowing/alias/builtin/let.wgsl
rename to test/tint/shadowing/short_names/let.wgsl
diff --git a/test/tint/shadowing/alias/builtin/let.wgsl.expected.dxc.hlsl b/test/tint/shadowing/short_names/let.wgsl.expected.dxc.hlsl
similarity index 100%
rename from test/tint/shadowing/alias/builtin/let.wgsl.expected.dxc.hlsl
rename to test/tint/shadowing/short_names/let.wgsl.expected.dxc.hlsl
diff --git a/test/tint/shadowing/alias/builtin/let.wgsl.expected.fxc.hlsl b/test/tint/shadowing/short_names/let.wgsl.expected.fxc.hlsl
similarity index 100%
rename from test/tint/shadowing/alias/builtin/let.wgsl.expected.fxc.hlsl
rename to test/tint/shadowing/short_names/let.wgsl.expected.fxc.hlsl
diff --git a/test/tint/shadowing/alias/builtin/let.wgsl.expected.glsl b/test/tint/shadowing/short_names/let.wgsl.expected.glsl
similarity index 100%
rename from test/tint/shadowing/alias/builtin/let.wgsl.expected.glsl
rename to test/tint/shadowing/short_names/let.wgsl.expected.glsl
diff --git a/test/tint/shadowing/alias/builtin/let.wgsl.expected.msl b/test/tint/shadowing/short_names/let.wgsl.expected.msl
similarity index 100%
rename from test/tint/shadowing/alias/builtin/let.wgsl.expected.msl
rename to test/tint/shadowing/short_names/let.wgsl.expected.msl
diff --git a/test/tint/shadowing/alias/builtin/let.wgsl.expected.spvasm b/test/tint/shadowing/short_names/let.wgsl.expected.spvasm
similarity index 100%
rename from test/tint/shadowing/alias/builtin/let.wgsl.expected.spvasm
rename to test/tint/shadowing/short_names/let.wgsl.expected.spvasm
diff --git a/test/tint/shadowing/alias/builtin/let.wgsl.expected.wgsl b/test/tint/shadowing/short_names/let.wgsl.expected.wgsl
similarity index 100%
rename from test/tint/shadowing/alias/builtin/let.wgsl.expected.wgsl
rename to test/tint/shadowing/short_names/let.wgsl.expected.wgsl
diff --git a/test/tint/shadowing/alias/builtin/param.wgsl b/test/tint/shadowing/short_names/param.wgsl
similarity index 100%
rename from test/tint/shadowing/alias/builtin/param.wgsl
rename to test/tint/shadowing/short_names/param.wgsl
diff --git a/test/tint/shadowing/alias/builtin/param.wgsl.expected.dxc.hlsl b/test/tint/shadowing/short_names/param.wgsl.expected.dxc.hlsl
similarity index 100%
rename from test/tint/shadowing/alias/builtin/param.wgsl.expected.dxc.hlsl
rename to test/tint/shadowing/short_names/param.wgsl.expected.dxc.hlsl
diff --git a/test/tint/shadowing/alias/builtin/param.wgsl.expected.fxc.hlsl b/test/tint/shadowing/short_names/param.wgsl.expected.fxc.hlsl
similarity index 100%
rename from test/tint/shadowing/alias/builtin/param.wgsl.expected.fxc.hlsl
rename to test/tint/shadowing/short_names/param.wgsl.expected.fxc.hlsl
diff --git a/test/tint/shadowing/alias/builtin/param.wgsl.expected.glsl b/test/tint/shadowing/short_names/param.wgsl.expected.glsl
similarity index 100%
rename from test/tint/shadowing/alias/builtin/param.wgsl.expected.glsl
rename to test/tint/shadowing/short_names/param.wgsl.expected.glsl
diff --git a/test/tint/shadowing/alias/builtin/param.wgsl.expected.msl b/test/tint/shadowing/short_names/param.wgsl.expected.msl
similarity index 100%
rename from test/tint/shadowing/alias/builtin/param.wgsl.expected.msl
rename to test/tint/shadowing/short_names/param.wgsl.expected.msl
diff --git a/test/tint/shadowing/alias/builtin/param.wgsl.expected.spvasm b/test/tint/shadowing/short_names/param.wgsl.expected.spvasm
similarity index 100%
rename from test/tint/shadowing/alias/builtin/param.wgsl.expected.spvasm
rename to test/tint/shadowing/short_names/param.wgsl.expected.spvasm
diff --git a/test/tint/shadowing/alias/builtin/param.wgsl.expected.wgsl b/test/tint/shadowing/short_names/param.wgsl.expected.wgsl
similarity index 100%
rename from test/tint/shadowing/alias/builtin/param.wgsl.expected.wgsl
rename to test/tint/shadowing/short_names/param.wgsl.expected.wgsl
diff --git a/test/tint/shadowing/alias/builtin/var.wgsl b/test/tint/shadowing/short_names/var.wgsl
similarity index 100%
rename from test/tint/shadowing/alias/builtin/var.wgsl
rename to test/tint/shadowing/short_names/var.wgsl
diff --git a/test/tint/shadowing/alias/builtin/var.wgsl.expected.dxc.hlsl b/test/tint/shadowing/short_names/var.wgsl.expected.dxc.hlsl
similarity index 100%
rename from test/tint/shadowing/alias/builtin/var.wgsl.expected.dxc.hlsl
rename to test/tint/shadowing/short_names/var.wgsl.expected.dxc.hlsl
diff --git a/test/tint/shadowing/alias/builtin/var.wgsl.expected.fxc.hlsl b/test/tint/shadowing/short_names/var.wgsl.expected.fxc.hlsl
similarity index 100%
rename from test/tint/shadowing/alias/builtin/var.wgsl.expected.fxc.hlsl
rename to test/tint/shadowing/short_names/var.wgsl.expected.fxc.hlsl
diff --git a/test/tint/shadowing/alias/builtin/var.wgsl.expected.glsl b/test/tint/shadowing/short_names/var.wgsl.expected.glsl
similarity index 100%
rename from test/tint/shadowing/alias/builtin/var.wgsl.expected.glsl
rename to test/tint/shadowing/short_names/var.wgsl.expected.glsl
diff --git a/test/tint/shadowing/alias/builtin/var.wgsl.expected.msl b/test/tint/shadowing/short_names/var.wgsl.expected.msl
similarity index 100%
rename from test/tint/shadowing/alias/builtin/var.wgsl.expected.msl
rename to test/tint/shadowing/short_names/var.wgsl.expected.msl
diff --git a/test/tint/shadowing/alias/builtin/var.wgsl.expected.spvasm b/test/tint/shadowing/short_names/var.wgsl.expected.spvasm
similarity index 100%
rename from test/tint/shadowing/alias/builtin/var.wgsl.expected.spvasm
rename to test/tint/shadowing/short_names/var.wgsl.expected.spvasm
diff --git a/test/tint/shadowing/alias/builtin/var.wgsl.expected.wgsl b/test/tint/shadowing/short_names/var.wgsl.expected.wgsl
similarity index 100%
rename from test/tint/shadowing/alias/builtin/var.wgsl.expected.wgsl
rename to test/tint/shadowing/short_names/var.wgsl.expected.wgsl