[wip] Add Windows support for cppyy#605
[wip] Add Windows support for cppyy#605mcbarton wants to merge 41 commits intocompiler-research:mainfrom
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #605 +/- ##
=======================================
Coverage 77.66% 77.66%
=======================================
Files 9 9
Lines 3743 3743
=======================================
Hits 2907 2907
Misses 836 836 🚀 New features to boost your workflow:
|
mcbarton
force-pushed
the
Add-Windows-support-cppyy
branch
from
c50b9e8 to
4467fca
Compare
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
| #if defined(_MSC_VER) | ||
| #pragma message("#include <clang/Interpreter/CppInterOp.h> is deprecated; use #include <CppInterOp/CppInterOp.h") | ||
| #else | ||
| #warning \ |
There was a problem hiding this comment.
warning: "#include <clang/Interpreter/CppInterOp.h> is deprecated; use #include <CppInterOp/CppInterOp.h" [clang-diagnostic-#warnings]
#warning \
^| #if defined(_MSC_VER) | ||
| #pragma message("#include <clang/Interpreter/CppInterOp.h> is deprecated; use #include <CppInterOp/CppInterOp.h") | ||
| #else | ||
| #warning \ |
There was a problem hiding this comment.
warning: #warning is a C++23 extension [clang-diagnostic-pedantic]
#warning \
^
mcbarton
force-pushed
the
Add-Windows-support-cppyy
branch
from
a5a2914 to
76a9d3e
Compare
|
clang-tidy review says "All clean, LGTM! 👍" |
4 similar comments
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
10 similar comments
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
mcbarton
force-pushed
the
Add-Windows-support-cppyy
branch
from
6a61704 to
2c2d652
Compare
|
clang-tidy review says "All clean, LGTM! 👍" |
|
I may revisit this in the future, but I will do it in a fresh PR |
1 participant
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.
Testing
Please describe the test(s) that you added and ran to verify your changes.
Checklist