[PATCH] D145131: [Arm][AArch64] Setting IsX18ReservedByDefault() to true for Unknown OSes

Murali Vijayaraghavan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 2 07:57:32 PST 2023


vmurali added a comment.

In D145131#4164594 <https://reviews.llvm.org/D145131#4164594>, @Benoit wrote:

> Thank you for the explanation regarding the current constraint that "unknown" includes in particular "bare metal".  Do you think that it would make sense going forward to start distinguishing between "bare metal" and "unknown"? After all, knowing that the target is "bare metal" is information in its own right, and here it enables using x18.

+1 to creating such a distinction


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D145131/new/

https://reviews.llvm.org/D145131



More information about the llvm-commits mailing list