The patch looks fine, but an explanation like "really a tiny thing" doesn't work. You have to explain why the patch is needed? Which versions are tested? And so on...
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Based on my month or so usage of the LLVM compiler infrastructure mixing of LLVM and other compiler tools does NOT always work.
As such this patch tries to use the LLVM resource compiler when using CLANG on windows in a similar way to the following change:
The patch looks fine, but an explanation like "really a tiny thing" doesn't work. You have to explain why the patch is needed? Which versions are tested? And so on...
Can we get an explanation?
Explanation?
Based on my month or so usage of the LLVM compiler infrastructure mixing of LLVM and other compiler tools does NOT always work.
As such this patch tries to use the LLVM resource compiler when using CLANG on windows in a similar way to the following change:
https://svn.code.sf.net/p/codeblocks/code/trunk@8712
https://github.com/acotty/codeblocks_sfmirror/commit/50e86470c11c0b0468b2ff2008eedd3ced7ba615
Once ticket [1285] (Windows C::B build errors with MSYS2 Clangw64) I will then be able to test this change if testing is required.