writers: Use the new sem::Module::DependencyOrderedDeclarations
As the resolver currently enforces in-order declarations, this does not
change the declaration order from iterating over the
ast::Module::GlobalDeclarations.
The MSL backend has been changed to use the
sem::Module::DependencyOrderedDeclarations list instead of looping over
different declaration types separately.
Bug: tint:1266
Change-Id: I698d612032285311017bfceab3c42adae1928a0e
Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/79767
Reviewed-by: James Price <jrprice@google.com>
Kokoro: Kokoro <noreply+kokoro@google.com>
diff --git a/test/array/assign_to_function_var.wgsl.expected.msl b/test/array/assign_to_function_var.wgsl.expected.msl
index 7abc32d..9d12733 100644
--- a/test/array/assign_to_function_var.wgsl.expected.msl
+++ b/test/array/assign_to_function_var.wgsl.expected.msl
@@ -4,18 +4,10 @@
struct tint_array_wrapper {
/* 0x0000 */ int4 arr[4];
};
+
struct S {
/* 0x0000 */ tint_array_wrapper arr;
};
-struct tint_array_wrapper_3 {
- int arr[2];
-};
-struct tint_array_wrapper_2 {
- tint_array_wrapper_3 arr[3];
-};
-struct tint_array_wrapper_1 {
- tint_array_wrapper_2 arr[4];
-};
tint_array_wrapper ret_arr() {
tint_array_wrapper const tint_symbol = {.arr={}};
@@ -27,6 +19,18 @@
return tint_symbol_1;
}
+struct tint_array_wrapper_3 {
+ int arr[2];
+};
+
+struct tint_array_wrapper_2 {
+ tint_array_wrapper_3 arr[3];
+};
+
+struct tint_array_wrapper_1 {
+ tint_array_wrapper_2 arr[4];
+};
+
void foo(tint_array_wrapper src_param, thread tint_array_wrapper* const tint_symbol_3, threadgroup tint_array_wrapper* const tint_symbol_4, const constant S* const tint_symbol_5, device S* const tint_symbol_6) {
tint_array_wrapper src_function = {};
tint_array_wrapper dst = {};
