Revert "[test] Report error when inferior test processes exit with a non-zero code"

This reverts r275782.

The problem with the commit is that it reports an additional "exit (1)" error for every file
containing a failing test, which is far more than I had intended to do. I'll need to come up with
a more fine-grained way of achieving the result.

llvm-svn: 275791

1 files changed, 17 insertions, 8 deletions

diff --git a/lldb/packages/Python/lldbsuite/test/test_runner/process_control.py b/lldb/packages/Python/lldbsuite/test/test_runner/process_control.py
index 5f693c1fa57..a7e639e4b8b 100644
--- a/lldb/packages/Python/lldbsuite/test/test_runner/process_control.py
+++ b/lldb/packages/Python/lldbsuite/test/test_runner/process_control.py
@@ -246,25 +246,33 @@ class ProcessHelper(object):
     def is_exceptional_exit(self, popen_status):
         """Returns whether the program exit status is exceptional.
 
-        Returns whether the return code from a Popen process is exceptional.
+        Returns whether the return code from a Popen process is exceptional
+        (e.g. signals on POSIX systems).
+
+        Derived classes should override this if they can detect exceptional
+        program exit.
 
         @return True if the given popen_status represents an exceptional
         program exit; False otherwise.
         """
-        return popen_status != 0
+        return False
 
     def exceptional_exit_details(self, popen_status):
         """Returns the normalized exceptional exit code and a description.
 
         Given an exceptional exit code, returns the integral value of the
-        exception and a description for the result.
+        exception (e.g. signal number for POSIX) and a description (e.g.
+        signal name on POSIX) for the result.
+
+        Derived classes should override this if they can detect exceptional
+        program exit.
 
-        Derived classes can override this if they want to want custom
-        exceptional exit code handling.
+        It is fine to not implement this so long as is_exceptional_exit()
+        always returns False.
 
         @return (normalized exception code, symbolic exception description)
         """
-        return (popen_status, "exit")
+        raise Exception("exception_exit_details() called on unsupported class")
 
 
 class UnixProcessHelper(ProcessHelper):
@@ -389,6 +397,9 @@ class UnixProcessHelper(ProcessHelper):
     def soft_terminate_signals(self):
         return [signal.SIGQUIT, signal.SIGTERM]
 
+    def is_exceptional_exit(self, popen_status):
+        return popen_status < 0
+
     @classmethod
     def _signal_names_by_number(cls):
         return dict(
@@ -396,8 +407,6 @@ class UnixProcessHelper(ProcessHelper):
             if v.startswith('SIG') and not v.startswith('SIG_'))
 
     def exceptional_exit_details(self, popen_status):
-        if popen_status >= 0:
-            return (popen_status, "exit")
         signo = -popen_status
         signal_names_by_number = self._signal_names_by_number()
         signal_name = signal_names_by_number.get(signo, "")