[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