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

[Dave Green via Phabricator via llvm-commits]

dmgreen added a comment.

I don't think this is the right approach to take. As far as I understand this will effect any aarch64-none-eabi targets that are compiling for bare metal (as the tests show), causing a performance hit to any bare metal application. They shouldn't have to pay the price if they don't actually use x18 for anything, which many will not.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145131