philipp.tomsich abandoned this revision.
philipp.tomsich added a comment.
I just saw that the last response was "unsubmitted".
Submitting and abandoning this revision (based on the discussion and the
previously unsubmitted comment from my end).
================
Comment at: llvm/include/llvm/TargetParser/AArch64TargetParser.h:311
inline constexpr ArchInfo ARMV8_4A = { VersionTuple{8, 4}, AProfile,
"armv8.4-a", "+v8.4a", (ARMV8_3A.DefaultExts | AArch64::AEK_DOTPROD)};
-inline constexpr ArchInfo ARMV8_5A = { VersionTuple{8, 5}, AProfile,
"armv8.5-a", "+v8.5a", (ARMV8_4A.DefaultExts)};
+inline constexpr ArchInfo ARMV8_5A = { VersionTuple{8, 5}, AProfile,
"armv8.5-a", "+v8.5a", (ARMV8_4A.DefaultExts | AArch64::AEK_SB |
AArch64::AEK_SSBS )};
constexpr unsigned BaseNoCrypto = ARMV8_5A.DefaultExts ^ AArch64::AEK_CRYPTO;
// 8.6 onwards has no AEK_CRYPTO
----------------
lenary wrote:
> According to the A-profile Arm ARM (version I.a):
> - FEAT_SB is mandatory from v8.5 onwards, so thank you for correcting this
> oversight.
> - There is no language about the same for FEAT_SSBS/FEAT_SSBS2. It was added
> to v8.0a by v8.5a, but it doesn't seem to have been made mandatory in v8.5a,
> as far as I can see. There is no language in section A2 (Architectural
> Extensions), or in the description of the `ID_AA64PFR1_EL1.SSBS` register
> about being mandatory from certain versions (other fields in this register do
> have this language).
I have now picked up I.a after I had previously referenced G.a.
Features added to earlier extensions (For ARMv8.5) lists FEAT_SBSS.
However, the Text under FEAT_SBSS/FEAT_SSBS2 no longer has the "This feature is
OPTIONAL in Armv8.0 implementations and mandatory in Armv8.5 implementations."
that was present in version G.a of the spec.
So it looks as if this was made optional again somewhere between G.a and I.a?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D142410/new/
https://reviews.llvm.org/D142410
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
