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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
