]> granicus.if.org Git - clang/commitdiff
[driver] Improve r141053 by only emitting the warning if the original input
authorChad Rosier <mcrosier@apple.com>
Tue, 4 Oct 2011 01:53:36 +0000 (01:53 +0000)
committerChad Rosier <mcrosier@apple.com>
Tue, 4 Oct 2011 01:53:36 +0000 (01:53 +0000)
was assembly.  Otherwise, something like -save-temps causes the integrated
assembler to warn.

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@141055 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Driver/Tools.cpp

index 08d95f51e207d7c9cb9b8bcc5daa7e29247e1eac..4762c650409d948bb12ed87a65f77cf5b833b825 100644 (file)
@@ -2318,10 +2318,20 @@ void ClangAs::ConstructJob(Compilation &C, const JobAction &JA,
   // Ignore explicit -force_cpusubtype_ALL option.
   (void) Args.hasArg(options::OPT_force__cpusubtype__ALL);
 
+  // Determine the original source input.
+  const Action *SourceAction = &JA;
+  while (SourceAction->getKind() != Action::InputClass) {
+    assert(!SourceAction->getInputs().empty() && "unexpected root action!");
+    SourceAction = SourceAction->getInputs()[0];
+  }
+
   // FIXME: Add -g support, once we have it.  For now, emit a warning indicating
   // the integrated assembler doesn't support debug info.
-  if (Args.hasArg(options::OPT_g_Group)) {
-    getToolChain().getDriver().Diag(diag::warn_drv_no_debug_w_integrated_as);
+  if (SourceAction->getType() == types::TY_Asm ||
+      SourceAction->getType() == types::TY_PP_Asm) {
+    if (Args.hasArg(options::OPT_g_Group)) {
+      getToolChain().getDriver().Diag(diag::warn_drv_no_debug_w_integrated_as);
+    }
   }
 
   // FIXME: Add -static support, once we have it.