[PATCH] D21428: Fix feature RAS assigned to ARMv8.1 when it should be ARMv8.2

Roger Ferrer Ibanez via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 16 03:22:27 PDT 2016


rogfer01 created this revision.
rogfer01 added reviewers: olista01, t.p.northover, rengolin.
rogfer01 added subscribers: llvm-commits, SjoerdMeijer.
Herald added subscribers: rengolin, aemerson.

Revision 271670 (which was comitted on behalf of me by @sjoerdmeijer) had a mistake where RAS extensions where assigned to 8.1 rather than 8.2.

http://reviews.llvm.org/D21428

Files:
  include/llvm/Support/ARMTargetParser.def

Index: include/llvm/Support/ARMTargetParser.def
===================================================================
--- include/llvm/Support/ARMTargetParser.def
+++ include/llvm/Support/ARMTargetParser.def
@@ -89,11 +89,11 @@
 ARM_ARCH("armv8.1-a", AK_ARMV8_1A, "8.1-A", "v8.1a",
          ARMBuildAttrs::CPUArch::v8_A, FK_CRYPTO_NEON_FP_ARMV8,
          (ARM::AEK_SEC | ARM::AEK_MP | ARM::AEK_VIRT | ARM::AEK_HWDIVARM |
-          ARM::AEK_HWDIV | ARM::AEK_DSP | ARM::AEK_CRC | ARM::AEK_RAS))
+          ARM::AEK_HWDIV | ARM::AEK_DSP | ARM::AEK_CRC))
 ARM_ARCH("armv8.2-a", AK_ARMV8_2A, "8.2-A", "v8.2a",
          ARMBuildAttrs::CPUArch::v8_A, FK_CRYPTO_NEON_FP_ARMV8,
          (ARM::AEK_SEC | ARM::AEK_MP | ARM::AEK_VIRT | ARM::AEK_HWDIVARM |
-          ARM::AEK_HWDIV | ARM::AEK_DSP | ARM::AEK_CRC))
+          ARM::AEK_HWDIV | ARM::AEK_DSP | ARM::AEK_CRC | ARM::AEK_RAS))
 ARM_ARCH("armv8-m.base", AK_ARMV8MBaseline, "8-M.Baseline", "v8m.base",
           ARMBuildAttrs::CPUArch::v8_M_Base, FK_NONE, ARM::AEK_HWDIV)
 ARM_ARCH("armv8-m.main", AK_ARMV8MMainline, "8-M.Mainline", "v8m.main",


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D21428.60955.patch
Type: text/x-patch
Size: 1095 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160616/646018c4/attachment.bin>


More information about the llvm-commits mailing list