-
Notifications
You must be signed in to change notification settings - Fork 125
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
Thunks/gen: Add support for compiling against clang 19 #4193
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.
nice cleanup :)
Unit tests ( diff --git unittests/ThunkLibs/generator.cpp unittests/ThunkLibs/generator.cpp
index d3fc9c7c0..38eadbfd6 100644
--- unittests/ThunkLibs/generator.cpp
+++ unittests/ThunkLibs/generator.cpp
@@ -417,7 +417,7 @@ TEST_CASE_METHOD(Fixture, "VersionedLibrary") {
CHECK_THAT(output, matches(callExpr(callee(functionDecl(hasName("dlopen"))), hasArgument(0, stringLiteral().bind("libname"))))
.check_binding(
- "libname", +[](const clang::StringLiteral* lit) { return lit->getString().endswith(".so.123"); }));
+ "libname", +[](const clang::StringLiteral* lit) { return lit->getString().ends_with(".so.123"); }));
}
TEST_CASE_METHOD(Fixture, "FunctionPointerViaType") { Other than that, tested and confirmed to produce working error messages equivalent to older LLVM versions. Well done! |
e3a8b6f
to
b2c6b2f
Compare
Apparently the CI runners still use LLVM 14 🙃 EDIT: I answered too late, you already fixed it |
b2c6b2f
to
00f5982
Compare
00f5982
to
426569d
Compare
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.
Perfect, thanks again!
No description provided.