[PATCH] D156784: [AArch64][PAC] Declare FPAC subtarget feature

Anatoly Trosinenko via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 11 05:39:37 PDT 2023


atrosinenko added a comment.

As discussed in D156716 <https://reviews.llvm.org/D156716>, it is not clear if I have to add FeatureFPAC to every relevant CPU. Maybe it is worth conservatively assuming that this feature should only be enabled manually by the user as a precaution against "I have CPU core X but it is not listed, so let's use cpu=Y because X supports all the instructions supported by Y //(but not FEAT_FPAC)//" - that would not cause any obvious run-time crashes under normal operation, but would make the code less secure.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D156784



More information about the llvm-commits mailing list