[PATCH] [AArch64] Add workaround for Cortex-A53 erratum (835769)

Renato Golin renato.golin at linaro.org
Fri Oct 10 02:39:26 PDT 2014


Assuming the LLVM part is good, this looks good to me, too. Thanks!

================
Comment at: include/clang/Driver/Options.td:1169
@@ -1168,1 +1168,3 @@
 
+def mfix_cortex_a53_835769 : Flag<["-"], "mfix-cortex-a53-835769">,
+  Group<m_aarch64_Features_Group>,
----------------
That is a horrible name for a flag, but I don't have a better one. :)

================
Comment at: test/Driver/aarch64-fix-cortex-a53-835769-cg.c:18
@@ +17,3 @@
+// CHECK-YES-NEXT: nop
+// CHECK-NO-NEXT-NOT: nop
+// CHECK-NEXT: madd
----------------
It's amazing how this works out of the box... :)

http://reviews.llvm.org/D5722






More information about the cfe-commits mailing list