Bug 1539814 - An initial pair of try presets for SpiderMonkey r=jandem,ahal - sm-shell: Selects shell only test cases that shouldn't require a full browser build. - sm-all: Selects test cases that may require a full browser build. Differential Revision: https://phabricator.services.mozilla.com/D28994

Bug 1550807 - Load the baselinescript into scratch1 before calling emitTraceLoggerResume r=jandem Tracelogger is no longer functioning properly because of bad script event ids. The baselinescript load into scratch1 was accidentally removed leading garbage script ids to be passed into emitTraceLoggerResume. This fix aims to simply reload the correct value back into scratch1 before calling tracelogger. Differential Revision: https://phabricator.services.mozilla.com/D30680

Bug 1548611: Update RNewArray::recover to match CodeGenerator::visitNewArrayCallVM r=jandem Differential Revision: https://phabricator.services.mozilla.com/D30393

Bug 1549340 - Part 2: Use union instead of reinterpret_cast to initialize const char* with symbol for JSPropertySpec.name. r=jandem Differential Revision: https://phabricator.services.mozilla.com/D30493

Bug 1549340 - Part 2: Use union instead of reinterpret_cast to initialize const char* with symbol for JSPropertySpec.name. r=jandem Differential Revision: https://phabricator.services.mozilla.com/D30493

Bug 1550679 - Remove duplicate typedef in ARM assembler r=jandem Differential Revision: https://phabricator.services.mozilla.com/D30617

Bug 1549667 - [mips] skip tests which need --spectre-XXX on. r=jandem Reviewers: jandem, lth Reviewed By: jandem Subscribers: jandem Bug #: 1549667 Differential Revision: https://phabricator.services.mozilla.com/D30172

Bug 1549667 - [mips64] Adding missing 'breaks' to 'switch' statements. r=jandem Reviewers: jandem Reviewed By: jandem Subscribers: jandem Bug #: 1549667 Differential Revision: https://phabricator.services.mozilla.com/D30167

Bug 1549667 - [mips] Add function cmp32Move{32,ptr}. r=jandem Reviewers: jandem Reviewed By: jandem Subscribers: jandem Bug #: 1549667 Differential Revision: https://phabricator.services.mozilla.com/D30165

Bug 1395509 - Track malloc memory associated with strings r=jandem This adds memory tracking for string contents while leaving the current scheme in place for the time being. Differential Revision: https://phabricator.services.mozilla.com/D30517

Bug 1549278 - [mips] Wasm/Ion: generate better code for wasmSelect in some cases. r=jandem port https://phabricator.services.mozilla.com/D27071 to mips. Differential Revision: https://phabricator.services.mozilla.com/D29999

Bug 1548793 - Add some more assertions when creating ProxyObjects r=jandem? Differential Revision: https://phabricator.services.mozilla.com/D29806

Bug 1549340 - Use macro instead of static method to pass reinterpret_cast to constexpr. r=jandem Differential Revision: https://phabricator.services.mozilla.com/D30158

Bug 1549396 - Disable flaky GC test. r=jandem Differential Revision: https://phabricator.services.mozilla.com/D30058

Bug 1527902 - Enable javascript.options.bigint by default r=jandem,kmag Differential Revision: https://phabricator.services.mozilla.com/D29182

Bug 1548305. Fix incorrect Maybe<JSAutoRealm> use in Element::Animate. r=jandem We just want to enter the Realm unconditionally. Also, we can get the global from our GlobalObject. Differential Revision: https://phabricator.services.mozilla.com/D29580

Bug 1547565 - Remove AtomStateEntry and use ReadBarriered<JSAtom*> instead in the atoms table r=jandem Differential Revision: https://phabricator.services.mozilla.com/D29179

Bug 1546778 - Improve commenting in visitObjectGroupDispatch r=jandem DONTBUILD Differential Revision: https://phabricator.services.mozilla.com/D28722

Bug 1547203 - Don't expose CTypes with --fuzzing-safe in JS shell. r=jandem Differential Revision: https://phabricator.services.mozilla.com/D28961

Bug 1521158: Robustify cache-line invalidations on AARCH64. r=jandem

Bug 1544364: Throw an error when calling the transplant test-function on a sandbox global. r=jandem Differential Revision: https://phabricator.services.mozilla.com/D28475

Bug 1546376 - Part 3: Use Max() instead of duplicating calls. r=jandem Depends on D28482 Differential Revision: https://phabricator.services.mozilla.com/D28483

Bug 1546376 - Part 2: Don't expose PRMJ_FormatTime and PRMJTime when not used. r=jandem They are only used in non-Intl resp. system ICU builds in jsdate.cpp. Depends on D28481 Differential Revision: https://phabricator.services.mozilla.com/D28482

Bug 1546376 - Part 1: Remove Stopwatch.h. r=jandem Differential Revision: https://phabricator.services.mozilla.com/D28481

Bug 1532286 - Wasm/Ion: generate better code for wasmSelect in some cases. r=jandem. This patch creates better code, when compiling Wasm via Ion, for 32-bit integer selects which are guarded by a 32-bit integer compare. No other type combinations are optimised at present. Changes: * LIR-shared.h: Add new LIR node LWasmCompareAndSelect, holding the values to compare, the values to select over, and the comparison details. * CanEmitCompareAtUses: - Rewrite (transform mechanically) remove hard-to-reason-about control flow (a loop which probably only iterates once, and a boolean control variable). These are both removed and replaced by obvious straight-line code. - Also allow deferred emission when there is a single user and it is a WasmSelect. * LIRGenerator::visitWasmSelect for both arm and x86-shared: - All target specific variants have been removed and replaced by a shared version. - If the condition is a compare, marked 'may be emitted at use point', and has the correct types, bundle up the comparison arguments and values to be selected-over into a single LWasmCompareAndSelect node. * CodeGenerator::visitWasmCompareAndSelect for both arm and x86-shared: - All target specific variants have been removed and replaced by a shared version. - From an LWasmCompareAndSelect node, generate the desired optimal sequence: a compare immediately followed by a conditional move. * CodeGenerator::generateBody(), ifdef JS_JITSPEW, one liner ridealong: print "# " before the LIR instruction name, so as to make reading IONFLAGS=codegen output easier. No (release-) functional change. * New test case select-int32.js. Apart from the removal of aarch64-specific lowering and codegen routines, none of the changes apply to aarch64, because wasm-via-Ion on aarch64 is not currently supported. Differential Revision: https://phabricator.services.mozilla.com/D27071

Bug 1543211 - Store offsets instead of lengths in SharedScriptData r=jandem When describing multiple packed arrays that make up a SharedScriptData we previously saved the length of each array and computed the start of an array from the other of the others. Instead, this patch records the offset from |this| to the end of each array. This makes seeking to the start of an array simpler while still allowing computing the length of an array as the difference of its start and end offsets. This patch replaces the SharedScriptData::natoms/noteLength fields with offsets instead. Differential Revision: https://phabricator.services.mozilla.com/D26844

Bug 1543211 - Move integer fields from JSScript to SharedScriptData r=jandem The SharedScriptData allocation always exists so moving these immutable integer fields allows more sharing for memory savings. This adds an extra dereference in a few paths in the JITs but they are negligable. Differential Revision: https://phabricator.services.mozilla.com/D26816

Bug 1543211 - Hash raw bytes in ScriptBytecodeHasher r=jandem As more fields get added to SharedScriptData it becomes more problematic to check them in hashing function so this changes code to hash raw bytes of the immutable portion of SharedScriptData itself as well as its trailing arrays (which currently hash as bytes already). This redefines those data ranges in terms of mozilla::Span to make it simpler to work with. The ScriptBytecodeHasher is renamed to SharedScriptDataHasher to be clearer about its scope. The ScriptBytecodeHasher::Lookup struct is removed. The self-hash of a lookup is only computed once during lookup so it is not necessary for use to cache the result ourselves. Differential Revision: https://phabricator.services.mozilla.com/D26802

Bug 1544631 - Fix functions: branchTestBigInt, negPtr, generateVMWrapper on MIPS. r=jandem Fix functions: branchTestBigInt, negPtr, generateVMWrapper on MIPS. Differential Revision: https://phabricator.services.mozilla.com/D27658

Bug 1406146 - Use ShellWindowProxy in jsshell by default. r=jandem,jorendorff Depends on D26839 Differential Revision: https://phabricator.services.mozilla.com/D26840

Bug 1543589 - Allow pretenuring of promoted nursery objects while context is in atoms zone r=jandem Differential Revision: https://phabricator.services.mozilla.com/D27329

Bug 1543159 - TypeHashSet should use a non-reserved bit for marking elements to be moved. r=jandem Differential Revision: https://phabricator.services.mozilla.com/D26961

Bug 1479970 - Use early exits in AutoFrontendTraceLog. r=jandem Avoid any risk of running non-trivial line-number or error reporting code. Differential Revision: https://phabricator.services.mozilla.com/D27110

Bug 1479970 - Remove IonBuilder::traceLogger(). r=jandem This only had one use is inconsistent with other tracelogger users. Differential Revision: https://phabricator.services.mozilla.com/D27103

Bug 1542292 - Bump 64-bit executable quota to ~2gb (r=jandem) Differential Revision: https://phabricator.services.mozilla.com/D26329

Bug 1536695 - Part 2: Support infallible conversion for BigInt types. r=jandem Differential Revision: https://phabricator.services.mozilla.com/D26119

Bug 1536695 - Part 1: Avoid computing constant scalar-type when calling canConvertInfallibly. r=jandem The scalar-type matches TypeIDOfType<T>::id, which is a constant, so it's not necessary to recompute the type for each call to canConvertInfallibly. Differential Revision: https://phabricator.services.mozilla.com/D26117

Bug 1528829 - Add test case to check range analysis correctness. r=jandem Differential Revision: https://phabricator.services.mozilla.com/D21930

Bug 1542461. Flag realms of windows as dead once we unlink their nsGlobalWindowInner, so debugger's findAllGlobals won't hand them out to script. r=jandem Differential Revision: https://phabricator.services.mozilla.com/D26563

Bug 1406146 - Fix cross-compartment global issue in typedarary test. r=jandem This is an artifact of the same-compartment realm changes. The test confuses CCWs and WindowProxies. Differential Revision: https://phabricator.services.mozilla.com/D26847

Bug 1406146 - Add jsshell test for WindowProxy ICs. r=jandem Depends on D26837 Differential Revision: https://phabricator.services.mozilla.com/D26838

Bug 1406146 - Support a simple WindowProxy in the jsshell. r=jandem This adds an option to the 'newGlobal' shell command to create globals that have a simple WindowProxy. This is intended for testing code that distinguishes between GlobalObject and WindowProxy. No typical window behavior such as navigation will be supported. Differential Revision: https://phabricator.services.mozilla.com/D26837

Bug 1406146 - Fix handling of WindowProxy in jsshell testing functions. r=jandem This patch fixes a few cases in testing functions that do not correctly handling WindowProxy wrappers. This mostly involves passing the stopAtWindowProxy=false argument to unwrap functions. Differential Revision: https://phabricator.services.mozilla.com/D26836

Bug 1506902 - Change JS::EvaluateUtf8 to directly compile the provided UTF-8 data, without inflating through UTF-16. r=jandem

Bug 1541580 - Cleanup ProxyObject creation r=jandem Differential Revision: https://phabricator.services.mozilla.com/D26157

Bug 1535137 - Store inline-doubles as Values in bytecode r=jandem Differential Revision: https://phabricator.services.mozilla.com/D26622

Bug 1535137 - Store JSOP_DOUBLE literals inline r=jandem Replace the unaligned uint32_t index with the encoded double value. The double values are already being copied again before use, so an unaligned uint64_t load directly to double seems a more direct strategy. This also avoids needed to manage the allocation of the consts table. Differential Revision: https://phabricator.services.mozilla.com/D23396

Bug 1535137 - Use copyAndSwap in BytecodeUtil instead of manual endian mangement r=jandem Differential Revision: https://phabricator.services.mozilla.com/D26621

Bug 1542292 - Bump 64-bit executable quota to 2gb (r=jandem) Differential Revision: https://phabricator.services.mozilla.com/D26329

Bug 1542292 - Bump 64-bit executable quota to 2gb (r=jandem) Differential Revision: https://phabricator.services.mozilla.com/D26329