[PATCH] D14121: Thumb state not being passed through to LLVM triple when using clang -cc1as

Alexandros Lamprineas via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 27 10:48:10 PDT 2015


labrinea added a comment.

Is this revision valid after all? I am confused by Tim's comment. I did not see any regressions locally.


================
Comment at: lib/Driver/ToolChain.cpp:472
@@ -471,2 +471,3 @@
+    bool ThumbDefault = (ARM::parseArchProfile(Suffix) == ARM::PK_M) ||
       (Suffix.startswith("v7") && getTriple().isOSBinFormatMachO());
     // FIXME: this is invalid for WindowsCE
----------------
rengolin wrote:
> You could cache the profile and use it here, too.
I don't see any checks based on profile in this line.

================
Comment at: test/Driver/arm-ias-Wa.s:75
@@ +74,3 @@
+
+// RUN: %clang -target thumbv7m-none-eabi -c %s -### 2>&1 \
+// RUN:   | FileCheck -check-prefix=CHECK-M-PROFILE %s
----------------
rengolin wrote:
> You should also add "armv7m" and check that it defaults to Thumb, no?
It does default to thumb, if we are happy with this check I can add it.


http://reviews.llvm.org/D14121





More information about the cfe-commits mailing list