[llvm-commits] [llvm] r167849 - in /llvm/trunk: lib/Support/ErrorHandling.cpp test/Other/close-stderr.ll

Chad Rosier mcrosier at apple.com
Tue Nov 13 08:42:19 PST 2012


Author: mcrosier
Date: Tue Nov 13 10:42:19 2012
New Revision: 167849

URL: http://llvm.org/viewvc/llvm-project?rev=167849&view=rev
Log:
Revert 167755/167760. We don't want to emit crash diagnostics on command-line syntax errors.

Modified:
    llvm/trunk/lib/Support/ErrorHandling.cpp
    llvm/trunk/test/Other/close-stderr.ll

Modified: llvm/trunk/lib/Support/ErrorHandling.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Support/ErrorHandling.cpp?rev=167849&r1=167848&r2=167849&view=diff
==============================================================================
--- llvm/trunk/lib/Support/ErrorHandling.cpp (original)
+++ llvm/trunk/lib/Support/ErrorHandling.cpp Tue Nov 13 10:42:19 2012
@@ -81,10 +81,7 @@
   // files registered with RemoveFileOnSignal.
   sys::RunInterruptHandlers();
 
-  // When reporting a fatal error, exit with status 70.  For BSD systems this
-  // is defined as an internal software error.  This notifies the driver to
-  // report diagnostics information.
-  exit(70);
+  exit(1);
 }
 
 void llvm::llvm_unreachable_internal(const char *msg, const char *file,

Modified: llvm/trunk/test/Other/close-stderr.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Other/close-stderr.ll?rev=167849&r1=167848&r2=167849&view=diff
==============================================================================
--- llvm/trunk/test/Other/close-stderr.ll (original)
+++ llvm/trunk/test/Other/close-stderr.ll Tue Nov 13 10:42:19 2012
@@ -1,6 +1,6 @@
 ; RUN: sh -c 'opt --reject-this-option 2>&-; echo $?; opt -o /dev/null /dev/null 2>&-; echo $?;' \
 ; RUN:   | FileCheck %s
-; CHECK: {{^70$}}
+; CHECK: {{^1$}}
 ; CHECK: {{^0$}}
 ; XFAIL: vg_leak
 ; REQUIRES: shell





More information about the llvm-commits mailing list