1091298 - Hazard analysis should give clear error instead of mangling assembly

Status: RESOLVED FIXED

(Core :: JavaScript: GC, defect)

Description

[Steve Fink [:sfink] [:s:]]

13:13:32     INFO -  In file included from /builds/slave/b2g_m-in_l64-b2g-haz_dep-00000/build/objdir-gecko/js/src/Unified_cpp_js_src3.cpp:137:0:
13:13:32     INFO -  /builds/slave/b2g_m-in_l64-b2g-haz_dep-00000/build/gecko/js/src/jit/ExecutableAllocator.cpp: At global scope:
13:13:32     INFO -  /builds/slave/b2g_m-in_l64-b2g-haz_dep-00000/build/gecko/js/src/jit/ExecutableAllocator.cpp:90:1: internal compiler error: Aborted
13:13:32     INFO -  Please submit a full bug report,
13:13:32     INFO -  with preprocessed source if appropriate.
13:13:32     INFO -  See <http://source.android.com/source/report-bugs.html> for instructions.
13:13:33     INFO -  {standard input}: Assembler messages:
13:13:33     INFO -  {standard input}:2754122: Error: unexpected character `c' in type specifier
13:13:33     INFO -  {standard input}:2754122: Error: bad instruction `backend/transaction.cpp:535:SubmitTransaction:Assertion 102alse32failed.'

So it output something like "backend/transaction.cpp:535:SubmitTransaction:Assertion failed." when the plugin failed, and that made it into the asm output. That should be going to stderr if it isn't already, and if it is, stderr should not be going to the asm file.

Comment 1

[Steve Fink [:sfink] [:s:]]

Fixed during the latest round of sixgill fixes. Deployed.