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



_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to