[cfe-commits] r149890 - in /cfe/trunk/lib/Driver: Driver.cpp Tools.cpp

[Eric Christopher]

Author: echristo
Date: Mon Feb  6 13:13:09 2012
New Revision: 149890

URL: http://llvm.org/viewvc/llvm-project?rev=149890&view=rev
Log:
Update the command line here and update the comment, we're just going
to leave this as a debug only option for now.

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

Modified: cfe/trunk/lib/Driver/Driver.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/Driver.cpp?rev=149890&r1=149889&r2=149890&view=diff
==============================================================================
--- cfe/trunk/lib/Driver/Driver.cpp (original)
+++ cfe/trunk/lib/Driver/Driver.cpp Mon Feb  6 13:13:09 2012
@@ -868,9 +868,8 @@
 
         Actions.push_back(new DsymutilJobAction(Inputs, types::TY_dSYM));
 
-	// Verify the debug output if we're in assert mode.
-	// TODO: The verifier is noisy by default so put this under an
-	// option for now.
+        // Verify the debug information if we're in debug mode and '-verify'
+        // is passed.
 	#ifndef NDEBUG
 	if (Args.hasArg(options::OPT_verify)) {
 	  ActionList VerifyInputs;

Modified: cfe/trunk/lib/Driver/Tools.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/Tools.cpp?rev=149890&r1=149889&r2=149890&view=diff
==============================================================================
--- cfe/trunk/lib/Driver/Tools.cpp (original)
+++ cfe/trunk/lib/Driver/Tools.cpp Mon Feb  6 13:13:09 2012
@@ -4079,6 +4079,8 @@
 				       const char *LinkingOutput) const {
   ArgStringList CmdArgs;
   CmdArgs.push_back("--verify");
+  CmdArgs.push_back("--debug-info");
+  CmdArgs.push_back("--eh-frame");
 
   assert(Inputs.size() == 1 && "Unable to handle multiple inputs.");
   const InputInfo &Input = Inputs[0];