[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