Checkmate50

Results 17 comments of 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.