Checkmate50
Checkmate50
Sorry for the slow reply, I didn't notice your reply earlier (I've updated my email settings now). The full command line arguments I used were: `examples/heterogeneous-hello-world/shader.slang -o examples/heterogeneous-hello-world/shader2.cpp -target cpp...
I've experimented some more with the input, and found that this error only occurs when attempting to compile with the `-heterogeneous` flag and when the `__GPU_FOREACH(...)` code is uncommented. So...
Sounds good. I opened up the HLSL code and found this line is the source of the error: `computeMain_wrapper(_S6, _S4, _S5);`, though why this is part of compiling `computeMain` I'm...
Alright, so I went back in Git history, and found that some change made since September has caused the generated HLSL text to also include all the C++ code in...
So this is the commit that breaks things, by introducing the C++ into the compiled HLSL: https://github.com/shader-slang/slang/commit/ee5842a01cdb2bd6cd4acee7214666f180b95d84
So I narrowed it down to the deletion of [line 1484](https://github.com/shader-slang/slang/commit/ee5842a01cdb2bd6cd4acee7214666f180b95d84#diff-e9d9d1281f79ee0fed14ff718e315e6a3b2b6aa3584bcb7cf3fadfc09f2e9265L1484). It seems like there was a decision to generalize the target to not be restricted to CPP and CUDA,...
I'm working on debugging this issue, so I think I can push out a fix shortly. I'll let you know if I get stuck for some reason though; [this branch](https://github.com/Checkmate50/slang/tree/cuda)...
Sounds good; I think option (2) makes sense to me, I'll start working on implementing that. It should be straightforward to just output the same C++ code without the blob...
I think null is actually a useful construct to have since it shows up a lot in C++/TS programming. Thoughts on what type it should have (I think it should...
Related, It's critical that we update the tests in the `compiler` folder, since those didn't catch this issue.