-
Notifications
You must be signed in to change notification settings - Fork 128
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Enable llvm19, remove llvm8 support. #1233
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clang-tidy made some suggestions
@@ -126,7 +126,7 @@ namespace utils { | |||
DECLARE_CLONE_FN(SubstNonTypeTemplateParmExpr) | |||
DECLARE_CLONE_FN(CXXScalarValueInitExpr) | |||
DECLARE_CLONE_FN(ConstantExpr) | |||
CLAD_COMPAT_8_DECLARE_CLONE_FN(ValueStmt) | |||
DECLARE_CLONE_FN(ValueStmt) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
warning: no header providing "clang::ValueStmt" is directly included [misc-include-cleaner]
DECLARE_CLONE_FN(ValueStmt)
^
CLAD_COMPAT_ExprValueKind_R_or_PR_Value, | ||
ExprObjectKind::OK_Ordinary CLAD_COMPAT_CLANG9_MemberExpr_ExtraParams( | ||
NOUR_None)); | ||
CLAD_COMPAT_ExprValueKind_R_or_PR_Value, ExprObjectKind::OK_Ordinary, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
warning: no header providing "clang::ExprObjectKind" is directly included [misc-include-cleaner]
CLAD_COMPAT_ExprValueKind_R_or_PR_Value, ExprObjectKind::OK_Ordinary,
^
ExprObjectKind::OK_Ordinary CLAD_COMPAT_CLANG9_MemberExpr_ExtraParams( | ||
NOUR_None)); | ||
CLAD_COMPAT_ExprValueKind_R_or_PR_Value, ExprObjectKind::OK_Ordinary, | ||
NOUR_None); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
warning: no header providing "clang::NOUR_None" is directly included [misc-include-cleaner]
NOUR_None);
^
DAP, FD->getNameInfo(), | ||
/*TemplateArgs=*/nullptr, m_Context.BoundMemberTy, | ||
CLAD_COMPAT_ExprValueKind_R_or_PR_Value, | ||
ExprObjectKind::OK_Ordinary, NOUR_None); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
warning: no header providing "clang::ExprObjectKind" is directly included [misc-include-cleaner]
ExprObjectKind::OK_Ordinary, NOUR_None);
^
DAP, FD->getNameInfo(), | ||
/*TemplateArgs=*/nullptr, m_Context.BoundMemberTy, | ||
CLAD_COMPAT_ExprValueKind_R_or_PR_Value, | ||
ExprObjectKind::OK_Ordinary, NOUR_None); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
warning: no header providing "clang::NOUR_None" is directly included [misc-include-cleaner]
ExprObjectKind::OK_Ordinary, NOUR_None);
^
@@ -739,7 +739,11 @@ | |||
// FIXME: currently this doesn't print func<templates>(args...) while | |||
// dumping and only prints func(args...), we need to fix this. | |||
auto* FTD = dyn_cast<FunctionTemplateDecl>(R.getRepresentativeDecl()); | |||
#if CLANG_VERSION_MAJOR < 19 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
warning: no header providing "CLANG_VERSION_MAJOR" is directly included [misc-include-cleaner]
lib/Differentiator/VisitorBase.cpp:32:
- #include <numeric>
+ #include <clang/Basic/Version.h>
+ #include <numeric>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1233 +/- ##
==========================================
- Coverage 94.65% 94.64% -0.01%
==========================================
Files 51 51
Lines 8901 8890 -11
==========================================
- Hits 8425 8414 -11
Misses 476 476
|
Fixes #1112.