Skip to content
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

Merged
merged 1 commit into from
Feb 3, 2025
Merged

Enable llvm19, remove llvm8 support. #1233

merged 1 commit into from
Feb 3, 2025

Conversation

vgvassilev
Copy link
Owner

Fixes #1112.

Copy link
Contributor

@github-actions github-actions bot left a 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)
Copy link
Contributor

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,
Copy link
Contributor

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);
Copy link
Contributor

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);
Copy link
Contributor

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);
Copy link
Contributor

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
Copy link
Contributor

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>

Copy link

codecov bot commented Feb 3, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.64%. Comparing base (715f6d2) to head (a925b12).
Report is 1 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@            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              
Files with missing lines Coverage Δ
include/clad/Differentiator/Compatibility.h 85.48% <ø> (-2.52%) ⬇️
include/clad/Differentiator/StmtClone.h 88.88% <ø> (ø)
lib/Differentiator/CladUtils.cpp 94.28% <100.00%> (ø)
lib/Differentiator/DerivativeBuilder.cpp 99.60% <100.00%> (ø)
lib/Differentiator/DiffPlanner.cpp 98.44% <100.00%> (+<0.01%) ⬆️
lib/Differentiator/ReverseModeVisitor.cpp 95.62% <ø> (ø)
lib/Differentiator/StmtClone.cpp 64.63% <100.00%> (ø)
lib/Differentiator/VisitorBase.cpp 97.73% <100.00%> (ø)
tools/ClangBackendPlugin.cpp 50.00% <ø> (ø)
tools/ClangBackendPlugin.h 0.00% <ø> (ø)
... and 2 more
Files with missing lines Coverage Δ
include/clad/Differentiator/Compatibility.h 85.48% <ø> (-2.52%) ⬇️
include/clad/Differentiator/StmtClone.h 88.88% <ø> (ø)
lib/Differentiator/CladUtils.cpp 94.28% <100.00%> (ø)
lib/Differentiator/DerivativeBuilder.cpp 99.60% <100.00%> (ø)
lib/Differentiator/DiffPlanner.cpp 98.44% <100.00%> (+<0.01%) ⬆️
lib/Differentiator/ReverseModeVisitor.cpp 95.62% <ø> (ø)
lib/Differentiator/StmtClone.cpp 64.63% <100.00%> (ø)
lib/Differentiator/VisitorBase.cpp 97.73% <100.00%> (ø)
tools/ClangBackendPlugin.cpp 50.00% <ø> (ø)
tools/ClangBackendPlugin.h 0.00% <ø> (ø)
... and 2 more

@vgvassilev vgvassilev merged commit 92f3acc into master Feb 3, 2025
97 of 98 checks passed
@vgvassilev vgvassilev deleted the llvm19 branch February 3, 2025 21:18
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.

Support LLVM19
1 participant