r252175 - [ARM] Clang gives unintended warning message for 'mthumb' + M-profiles:

Alexandros Lamprineas via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 5 09:11:55 PST 2015


Author: alelab01
Date: Thu Nov  5 11:11:55 2015
New Revision: 252175

URL: http://llvm.org/viewvc/llvm-project?rev=252175&view=rev
Log:
[ARM] Clang gives unintended warning message for 'mthumb' + M-profiles:

$ clang --target=armv7m-none-eabi -mthumb -march=armv7-m -c test.c
clang-3.8: warning: argument unused during compilation: '-mthumb'

Differential Revision: http://reviews.llvm.org/D14384

Modified:
    cfe/trunk/lib/Driver/ToolChain.cpp

Modified: cfe/trunk/lib/Driver/ToolChain.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/ToolChain.cpp?rev=252175&r1=252174&r2=252175&view=diff
==============================================================================
--- cfe/trunk/lib/Driver/ToolChain.cpp (original)
+++ cfe/trunk/lib/Driver/ToolChain.cpp Thu Nov  5 11:11:55 2015
@@ -481,9 +481,8 @@ std::string ToolChain::ComputeLLVMTriple
       ArchName = "arm";
 
     // Assembly files should start in ARM mode, unless arch is M-profile.
-    if (IsMProfile || (InputType != types::TY_PP_Asm &&
-         Args.hasFlag(options::OPT_mthumb, options::OPT_mno_thumb, ThumbDefault)))
-    {
+    if ((InputType != types::TY_PP_Asm && Args.hasFlag(options::OPT_mthumb,
+         options::OPT_mno_thumb, ThumbDefault)) || IsMProfile) {
       if (IsBigEndian)
         ArchName = "thumbeb";
       else




More information about the cfe-commits mailing list