mirror of
https://github.com/encounter/dawn-cmake.git
synced 2025-08-06 04:05:40 +00:00
0e22bdbae7
In order to avoid declaring too many function parameters, we previously modified this transform to redeclare private variables that are only used inside a single function as function-scope variables. This was broken as it meant that their values did not persist across multiple calls to the same function. Instead, wrap all private variables in a structure and pass it around as a pointer. Fixed: tint:1875 Change-Id: I83f5eb1071d57b9c6af56d6cf21b3a32c6e94260 Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/124800 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Commit-Queue: James Price <jrprice@google.com>
46 lines
1.0 KiB
Plaintext
46 lines
1.0 KiB
Plaintext
#include <metal_stdlib>
|
|
|
|
using namespace metal;
|
|
struct tint_private_vars_struct {
|
|
bool tint_discarded;
|
|
};
|
|
|
|
int f(int x, thread tint_private_vars_struct* const tint_private_vars) {
|
|
if ((x == 10)) {
|
|
(*(tint_private_vars)).tint_discarded = true;
|
|
}
|
|
return x;
|
|
}
|
|
|
|
struct tint_symbol_2 {
|
|
int3 x [[user(locn1)]] [[flat]];
|
|
};
|
|
|
|
struct tint_symbol_3 {
|
|
int value [[color(2)]];
|
|
};
|
|
|
|
int tint_symbol_inner(int3 x, thread tint_private_vars_struct* const tint_private_vars) {
|
|
int y = x[0];
|
|
while (true) {
|
|
int const r = f(y, tint_private_vars);
|
|
if ((r == 0)) {
|
|
break;
|
|
}
|
|
}
|
|
return y;
|
|
}
|
|
|
|
fragment tint_symbol_3 tint_symbol(tint_symbol_2 tint_symbol_1 [[stage_in]]) {
|
|
thread tint_private_vars_struct tint_private_vars = {};
|
|
tint_private_vars.tint_discarded = false;
|
|
int const inner_result = tint_symbol_inner(tint_symbol_1.x, &(tint_private_vars));
|
|
tint_symbol_3 wrapper_result = {};
|
|
wrapper_result.value = inner_result;
|
|
if (tint_private_vars.tint_discarded) {
|
|
discard_fragment();
|
|
}
|
|
return wrapper_result;
|
|
}
|
|
|