[PATCH] [ARM] Improve build attributes emission

Richard Barton richard.barton.arm at gmail.com
Mon Oct 7 08:37:08 PDT 2013


  Two small comments. Happy for you to commit with those changes.


================
Comment at: lib/Target/ARM/ARMAsmPrinter.cpp:745
@@ +744,3 @@
+                                            const ARMSubtarget *Subtarget) {
+  if (CPU == "xscale")
+    return ARMBuildAttrs::v5TEJ;
----------------
I think you should put a comment in to say why xscale/v5TEJ is treated as a special case.

================
Comment at: test/CodeGen/ARM/2010-09-29-mc-asm-header-test.ll:134
@@ +133,3 @@
+; CORTEX-R5:  .eabi_attribute 9, 2
+; CORTEX-R5:  .eabi_attribute 10, 3
+; CORTEX-R5:  .fpu vfpv3
----------------
VFPv3 needs the same fix as VFPv4 for _D16 version. This value then becomes 4.


http://llvm-reviews.chandlerc.com/D1811



More information about the llvm-commits mailing list