[PATCH] [ARM] Improve build attributes emission

Amara Emerson amara.emerson at arm.com
Thu Oct 3 07:37:19 PDT 2013



================
Comment at: lib/Target/ARM/ARMAsmPrinter.cpp:757
@@ +756,3 @@
+    return ARMBuildAttrs::v7;
+  else if (Subtarget->hasV6T2Ops())
+    return ARMBuildAttrs::v6T2;
----------------
Renato Golin wrote:
> Amara Emerson wrote:
> > Richard Barton wrote:
> > > It looks like this table will have to also take into account M-Profile to distinguish between v6-M and normal v6 architectures.
> > Agreed, I'll make this change.
> Isn't that covered by the Subtarget->isMClass() below?
Not quite, at the moment this change would emit arch=v6 and arch_profile=microcontroller. It should instead emit v6-M (or v6S-M).


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



More information about the llvm-commits mailing list