[PATCH] D145131: [Arm][AArch64] Setting IsX18ReservedByDefault() to true for Unknown OSes
Murali Vijayaraghavan via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Mar 1 17:05:08 PST 2023
vmurali created this revision.
vmurali added a reviewer: Benoit.
Herald added subscribers: hanchung, hiraditya, kristof.beyls.
Herald added a project: All.
vmurali requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.
Arm manual says: X18 is the platform register and is reserved for the use of platform ABIs. This is an additional temporary register on platforms that don't assign a special meaning to it.
Source: https://developer.arm.com/documentation/den0024/a/The-ABI-for-ARM-64-bit-Architecture/Register-use-in-the-AArch64-Procedure-Call-Standard/Parameters-in-general-purpose-registers
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D145131
Files:
llvm/lib/TargetParser/AArch64TargetParser.cpp
Index: llvm/lib/TargetParser/AArch64TargetParser.cpp
===================================================================
--- llvm/lib/TargetParser/AArch64TargetParser.cpp
+++ llvm/lib/TargetParser/AArch64TargetParser.cpp
@@ -97,7 +97,7 @@
bool AArch64::isX18ReservedByDefault(const Triple &TT) {
return TT.isAndroid() || TT.isOSDarwin() || TT.isOSFuchsia() ||
- TT.isOSWindows() || TT.isOHOSFamily();
+ TT.isOSWindows() || TT.isOHOSFamily() || TT.isOSUnknown();
}
// Allows partial match, ex. "v8a" matches "armv8a".
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D145131.501702.patch
Type: text/x-patch
Size: 543 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230302/c0c42184/attachment.bin>
More information about the llvm-commits
mailing list