Bug 1511618 - js/src/tests/lib/jittests.py check_output should detect rc 139 as a crash, r=jmaher.
authorBob Clary <bclary@bclary.com>
Mon, 31 Dec 2018 10:14:25 -0800
changeset 509312 2bdcac07076ada82ef762264eec42a2f1c76b7d1
parent 509311 8d7092d3884f197a5679a44ad6101621af72f825
child 509313 9af9e57b3a598dc2a1a6aa293344bfeee11acc9f
push id10547
push userffxbld-merge
push dateMon, 21 Jan 2019 13:03:58 +0000
treeherdermozilla-beta@24ec1916bffe [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjmaher
bugs1511618
milestone66.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1511618 - js/src/tests/lib/jittests.py check_output should detect rc 139 as a crash, r=jmaher.
js/src/tests/lib/jittests.py
--- a/js/src/tests/lib/jittests.py
+++ b/js/src/tests/lib/jittests.py
@@ -507,16 +507,21 @@ def check_output(out, err, rc, timed_out
             return True
 
         # When building with ASan enabled, ASan will convert the -11 returned
         # value to 1. As a work-around we look for the error output which
         # includes the crash reason.
         if rc == 1 and ("Hit MOZ_CRASH" in err or "Assertion failure:" in err):
             return True
 
+        # When running jittests on Android, SEGV results in a return code
+        # of 128+11=139.
+        if rc == 139:
+            return True
+
     if rc != test.expect_status:
         # Tests which expect a timeout check for exit code 6.
         # Sometimes 0 is returned on Windows for unknown reasons.
         # See bug 899697.
         if sys.platform in ['win32', 'cygwin'] and rc == 0:
             return True
 
         # Allow a non-zero exit code if we want to allow OOM, but only if we