Can we do Windows builds with clang? #124

New Issue

bvisness · 2023-09-17T19:21:09Z

bvisness commented

2023-09-17 19:21:09 +00:00

We already ask people to install clang in order to build samples, and the distinction between runtime builds and sample builds is not so clear. Furthermore, MSVC sucks across the board, what with late adoption of C features and terrible command-line setups with vcvarsall and whatnot.

Could we just build the runtime with clang on Windows instead? Then it's one less major dependency to install and probably much less pain and suffering by users.

We already ask people to install clang in order to build samples, and the distinction between runtime builds and sample builds is not so clear. Furthermore, MSVC sucks across the board, what with late adoption of C features and terrible command-line setups with vcvarsall and whatnot. Could we just build the runtime with clang on Windows instead? Then it's one less major dependency to install and probably much less pain and suffering by users.

bvisness added the

windows

label 2023-09-17 19:21:09 +00:00

bvisness commented

2023-09-19 02:18:28 +00:00

Unfortunately, throwing clang-cl at it did not work. @MartinFouilleul, this is probably something you'll need to look at since I'm not actually that familiar with everything Orca is using.

rdunnington commented

2023-09-19 03:44:29 +00:00

One complication is that if we want to use DirectX, MSVC has better support for it than other compilers. It's interfaces are all C++, and thanks to the unstable C++ ABI, other compilers can't use it directly afaik. We could probably try using it with #define CINTERFACE, but the DXC shader compiler doesn't support that yet: https://github.com/microsoft/DirectXShaderCompiler/issues/3162.

One complication is that if we want to use DirectX, MSVC has better support for it than other compilers. It's interfaces are all C++, and thanks to the unstable C++ ABI, other compilers can't use it directly afaik. We could probably try using it with `#define CINTERFACE`, but the DXC shader compiler doesn't support that yet: https://github.com/microsoft/DirectXShaderCompiler/issues/3162.

bvisness commented

2023-09-19 04:01:43 +00:00

Hm. Maybe I should just get CI working and sidestep the issue then.