[PATCH] D75044: [AArch64] __builtin_extract_return_addr for PAuth.

2020-04-08 Thread Daniel Kiss via Phabricator via cfe-commits
danielkiss marked 4 inline comments as done. danielkiss added inline comments. Comment at: llvm/include/llvm/CodeGen/ISDOpcodes.h:74 /// the parent's frame or return address, and so on. -FRAMEADDR, RETURNADDR, ADDROFRETURNADDR, SPONENTRY, +FRAMEADDR, RETURNADDR,

[PATCH] D75044: [AArch64] __builtin_extract_return_addr for PAuth.

2020-04-08 Thread Daniel Kiss via Phabricator via cfe-commits
danielkiss updated this revision to Diff 256069. danielkiss added a comment. Rebased and address review comments. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D75044/new/ https://reviews.llvm.org/D75044 Files: clang/lib/CodeGen/TargetInfo.cpp

[PATCH] D75044: [AArch64] __builtin_extract_return_addr for PAuth.

2020-04-07 Thread Momchil Velikov via Phabricator via cfe-commits
chill added inline comments. Comment at: clang/lib/CodeGen/TargetInfo.cpp:5151 + return Address; +} +llvm::Function *F = Can drop the extra braces here. Comment at: clang/lib/CodeGen/TargetInfo.cpp:5156 +llvm::CallInst *Call =

[PATCH] D75044: [AArch64] __builtin_extract_return_addr for PAuth.

2020-04-07 Thread Momchil Velikov via Phabricator via cfe-commits
chill added a comment. In D75044#1966997 , @chill wrote: > Needs a test in `clang/test` that `__builtin_extract_return_address` is > translated to `llvm.extractreturnaddress`. Nevermind, I'm blind. CHANGES SINCE LAST ACTION

[PATCH] D75044: [AArch64] __builtin_extract_return_addr for PAuth.

2020-04-07 Thread Momchil Velikov via Phabricator via cfe-commits
chill added a comment. Needs a test in `clang/test` that `__builtin_extract_return_address` is translated to `llvm.extractreturnaddress`. What if LLVM IR contains a call to `llvm.extractreturnaddress`, but the target is not AArch64? Comment at:

[PATCH] D75044: [AArch64] __builtin_extract_return_addr for PAuth.

2020-04-03 Thread Daniel Kiss via Phabricator via cfe-commits
danielkiss updated this revision to Diff 254921. danielkiss added a comment. Rebased. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D75044/new/ https://reviews.llvm.org/D75044 Files: clang/lib/CodeGen/TargetInfo.cpp clang/test/CodeGen/arm64-extractreturnaddress.c

[PATCH] D75044: [AArch64] __builtin_extract_return_addr for PAuth.

2020-02-26 Thread Daniel Kiss via Phabricator via cfe-commits
danielkiss updated this revision to Diff 246685. danielkiss added a reviewer: olista01. danielkiss added a comment. rebased and test is updated. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D75044/new/ https://reviews.llvm.org/D75044 Files: clang/lib/CodeGen/TargetInfo.cpp

[PATCH] D75044: [AArch64] __builtin_extract_return_addr for PAuth.

2020-02-25 Thread Daniel Kiss via Phabricator via cfe-commits
danielkiss marked 2 inline comments as done. danielkiss added inline comments. Comment at: clang/test/CodeGen/arm64-extractreturnaddress.c:15 +// CHECK: define dso_local i8* @bar() #0 { +// CHECK-NEXT:ret i8* inttoptr (i64 42 to i8*) +// CHECK-PAC: %1 = call i8*

[PATCH] D75044: [AArch64] __builtin_extract_return_addr for PAuth.

2020-02-24 Thread Daniel Kiss via Phabricator via cfe-commits
danielkiss created this revision. danielkiss added reviewers: ostannard, asl. Herald added subscribers: llvm-commits, cfe-commits, jdoerfert, hiraditya, kristof.beyls. Herald added projects: clang, LLVM. This change adds the support for __builtin_extract_return_addr for ARMv8.3A Pointer