Skip to content

Conversation

@agarny
Copy link
Contributor

@agarny agarny commented Jan 8, 2025

Fixes #368.

@agarny agarny force-pushed the issue368 branch 4 times, most recently from c2f7947 to cb3084e Compare January 8, 2025 19:42
@agarny agarny force-pushed the issue368 branch 16 times, most recently from a338128 to de2e566 Compare January 9, 2025 02:45
…our third-party libraries.

Indeed, this is because we build llvm-tblgen and clang-tblgen using a bash script, so to get env.DLLVMCLANG_LLVM_TABLEGEN and env.DLLVMCLANG_CLANG_TABLEGEN properly set, we need to stick to bash. If we were not to do that on Windows, then env.DLLVMCLANG_LLVM_TABLEGEN and env.DLLVMCLANG_CLANG_TABLEGEN would be split in two!? Indeed, CMake would eventually warn about ignoring "D:/a/libopencor/libopencor/build/.1.8/build/bin/llvm-tblgen.exe" and ".1.8/build/bin/clang-tblgen.exe"!?
@agarny agarny force-pushed the issue368 branch 6 times, most recently from ce18290 to 2d42c85 Compare January 9, 2025 06:43
@agarny agarny force-pushed the issue368 branch 2 times, most recently from b19bea3 to f140469 Compare January 9, 2025 11:07
@agarny
Copy link
Contributor Author

agarny commented Jan 15, 2025

Closing in favour of PR #372.

@agarny agarny closed this Jan 15, 2025
@agarny agarny deleted the issue368 branch January 15, 2025 06:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

(Re)add ARM support on Windows and Linux

1 participant