[llvm-branch-commits] [clang] [clang] Implement pointer authentication for C++ virtual functions, v-tables, and VTTs. (PR #93907)

Oliver Hunt via llvm-branch-commits llvm-branch-commits at lists.llvm.org
Fri May 31 02:06:12 PDT 2024


ojhunt wrote:

Build fix is trivial

`diff --git a/clang/lib/Sema/SemaDeclAttr.cpp b/clang/lib/Sema/SemaDeclAttr.cpp
index 82c4a3c86645..e9a867ff67ba 100644
--- a/clang/lib/Sema/SemaDeclAttr.cpp
+++ b/clang/lib/Sema/SemaDeclAttr.cpp
@@ -9246,7 +9246,7 @@ static void handleVTablePointerAuthentication(Sema &S, Decl *D,
       AL.setInvalid();
     }
 
-    if (!AL.isArgExpr(3) || !S.checkUInt32Argument(AL, AL.getArgAsExpr(3),
+    if (!AL.isArgExpr(3) || !checkUInt32Argument(S, AL, AL.getArgAsExpr(3),
                                                    customDiscriminationValue)) {
       S.Diag(AL.getLoc(), diag::err_invalid_custom_discrimination);
       AL.setInvalid();`
       
Just the original PR being merged is dates to earlier checkUInt32Argument API

https://github.com/llvm/llvm-project/pull/93907


More information about the llvm-branch-commits mailing list