[PATCH] [AArch64] Add workaround for Cortex-A53 erratum (835769)
Bradley Smith
bradley.smith at arm.com
Fri Oct 10 07:16:19 PDT 2014
================
Comment at: test/CodeGen/AArch64/aarch64-fix-cortex-a53-835769.ll:36
@@ +35,3 @@
+; CHECK-NOWORKAROUND-NEXT: madd
+; CHECK-BASIC-PASS-DISABLED-LABEL: f_load_madd_64:
+; CHECK-BASIC-PASS-DISABLED: ldr
----------------
rengolin wrote:
> If BASIC-PASS-DISABLED == NOWORKAROUND, why have two?
All of the below tests are very sensitive to scheduling, and indeed if you run all of the NOWORKAROUND tests for a57/generic/cyclone some of them will fail for this very reason.
Instead only a single test is done for BASIC-PASS-DISABLED just to check the pass doesn't run.
http://reviews.llvm.org/D5721
More information about the llvm-commits
mailing list