[wip] Add llvm 21 support

[mcbarton]

Description

Please include a summary of changes, motivation and context for this PR.

Fixes # (issue)

Type of change

Please tick all options which are relevant.

• Bug fix
• New feature
• Requires documentation updates

Testing

Please describe the test(s) that you added and ran to verify your changes.

Checklist

• I have read the contribution guide recently

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.83%. Comparing base (33a3e7e) to head (f8c7163).

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #672   +/-   ##
=======================================
  Coverage   79.83%   79.83%           
=======================================
  Files           9        9           
  Lines        3962     3962           
=======================================
  Hits         3163     3163           
  Misses        799      799           

Files with missing lines	Coverage Δ
lib/CppInterOp/Compatibility.h	92.03% <ø> (ø)
lib/CppInterOp/CppInterOp.cpp	88.04% <100.00%> (ø)

Files with missing lines	Coverage Δ
lib/CppInterOp/Compatibility.h	92.03% <ø> (ø)
lib/CppInterOp/CppInterOp.cpp	88.04% <100.00%> (ø)

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[github-actions]

clang-tidy made some suggestions

[github-actions]

clang-tidy review says "All clean, LGTM! 👍"

[github-actions]

clang-tidy review says "All clean, LGTM! 👍"

[github-actions]

clang-tidy review says "All clean, LGTM! 👍"

[github-actions]

clang-tidy review says "All clean, LGTM! 👍"

[github-actions]

clang-tidy review says "All clean, LGTM! 👍"

[github-actions]

clang-tidy review says "All clean, LGTM! 👍"

[mcbarton]

Current progress with adding compatibility for llvm 21. Can build on all platforms. Can pass all tests for native Windows, native Linux and Emscripten platform. MacOS errors for the following test on both arm and x86

1: [ RUN ] InterpreterTest.IncludePaths
1: LLVM ERROR: Invalid option set!

[github-actions]

clang-tidy review says "All clean, LGTM! 👍"

[github-actions]

clang-tidy review says "All clean, LGTM! 👍"

[mcbarton]

For the MacOS failures we apear to be triggering this line

CppInterOp/lib/CppInterOp/Paths.cpp

Line 161 in 82f08c6

if (E.IsFramework && E.Group != frontend::Angled)

. Not entirely sure what that error is catching at first glance, and why its not triggered for other platforms, but will investigate.

[github-actions]

clang-tidy review says "All clean, LGTM! 👍"

[mcbarton]

Something has happened to the llvm 21 release branch in the last 2 weeks to break the ScopeReflectionTest.GetNumBases test in CppInterOp. Also the Windows build of CppInterOp has broken somehow.

[github-actions]

clang-tidy review says "All clean, LGTM! 👍"

[anutosh491]

Hey @mcbarton ,

Are you working on this ?

I think I saw quite some errors errors while trying to build cppinterop against latest llvm 21 few days back.

[github-actions]

clang-tidy review says "All clean, LGTM! 👍"

[mcbarton]

Hey @mcbarton ,

Are you working on this ?

I think I saw quite some errors errors while trying to build cppinterop against latest llvm 21 few days back.

@anutosh491 With the exception of the Windows native build you can build CppInterOp against llvm 21 with the changes in this PR (Windows fails building llvm). The tests do not pass however. Given the changes in this PR are minimal, then the source of these errors are likely on the llvm side. I will update this PR soon to get rid of the Windows Emscripten cross compile patch since it is no longer needed, and to update the documentation. Not sure I have any spare time at the moment though to work out the source of the failing tests.