Skip to content

Commit 56b02ec

Browse files
authored
assertions cleanup, remove unnecessary work when they are disabled (#5991)
1 parent 6f874fa commit 56b02ec

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed

src/library.js

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1136,12 +1136,11 @@ LibraryManager.library = {
11361136
llvm_prefetch: function(){},
11371137

11381138
__assert_fail: function(condition, filename, line, func) {
1139-
ABORT = true;
1140-
throw 'Assertion failed: ' + Pointer_stringify(condition) + ', at: ' + [filename ? Pointer_stringify(filename) : 'unknown filename', line, func ? Pointer_stringify(func) : 'unknown function'] + ' at ' + stackTrace();
1139+
abort('Assertion failed: ' + Pointer_stringify(condition) + ', at: ' + [filename ? Pointer_stringify(filename) : 'unknown filename', line, func ? Pointer_stringify(func) : 'unknown function']);
11411140
},
11421141

11431142
__assert_func: function(filename, line, func, condition) {
1144-
throw 'Assertion failed: ' + (condition ? Pointer_stringify(condition) : 'unknown condition') + ', at: ' + [filename ? Pointer_stringify(filename) : 'unknown filename', line, func ? Pointer_stringify(func) : 'unknown function'] + ' at ' + stackTrace();
1143+
abort('Assertion failed: ' + (condition ? Pointer_stringify(condition) : 'unknown condition') + ', at: ' + [filename ? Pointer_stringify(filename) : 'unknown filename', line, func ? Pointer_stringify(func) : 'unknown function']);
11451144
},
11461145

11471146
$EXCEPTIONS: {

src/preamble.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1714,7 +1714,9 @@ if (!Math['trunc']) Math['trunc'] = function(x) {
17141714
};
17151715
Math.trunc = Math['trunc'];
17161716
#else // LEGACY_VM_SUPPORT
1717+
#if ASSERTIONS
17171718
assert(Math['imul'] && Math['fround'] && Math['clz32'] && Math['trunc'], 'this is a legacy browser, build with LEGACY_VM_SUPPORT');
1719+
#endif
17181720
#endif // LEGACY_VM_SUPPORT
17191721

17201722
var Math_abs = Math.abs;

0 commit comments

Comments
 (0)