[cfe-commits] r141127 - in /cfe/trunk: include/clang/Basic/DiagnosticDriverKinds.td lib/Driver/Tools.cpp

Chad Rosier mcrosier at apple.com
Tue Oct 4 15:35:50 PDT 2011


Author: mcrosier
Date: Tue Oct  4 17:35:48 2011
New Revision: 141127

URL: http://llvm.org/viewvc/llvm-project?rev=141127&view=rev
Log:
[driver] Improve r141053 by only emitting the warning if the original input
was assembly.  Otherwise, something like -save-temps causes the integrated
assembler to warn.

Modified:
    cfe/trunk/include/clang/Basic/DiagnosticDriverKinds.td
    cfe/trunk/lib/Driver/Tools.cpp

Modified: cfe/trunk/include/clang/Basic/DiagnosticDriverKinds.td
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/DiagnosticDriverKinds.td?rev=141127&r1=141126&r2=141127&view=diff
==============================================================================
--- cfe/trunk/include/clang/Basic/DiagnosticDriverKinds.td (original)
+++ cfe/trunk/include/clang/Basic/DiagnosticDriverKinds.td Tue Oct  4 17:35:48 2011
@@ -122,9 +122,6 @@
   "Objective-C garbage collection is not supported on this platform, ignoring '%0'">;
 def warn_drv_pch_not_first_include : Warning<
   "precompiled header '%0' was ignored because '%1' is not first '-include'">;
-def warn_drv_no_debug_w_integrated_as : Warning<
-  "ignoring unsupported -g option with integrated assembler">,
-  InGroup<DiagGroup<"unsupported-debug-with-integrated-as">>;
 
 def note_drv_command_failed_diag_msg : Note<
   "diagnostic msg: %0">;

Modified: cfe/trunk/lib/Driver/Tools.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/Tools.cpp?rev=141127&r1=141126&r2=141127&view=diff
==============================================================================
--- cfe/trunk/lib/Driver/Tools.cpp (original)
+++ cfe/trunk/lib/Driver/Tools.cpp Tue Oct  4 17:35:48 2011
@@ -2318,21 +2318,7 @@
   // 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 (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 -g support, once we have it.
 
   // FIXME: Add -static support, once we have it.
 





More information about the cfe-commits mailing list