André suggested some performance improvements that are worth addressing.
Bug 1688913 Comment 0 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
André suggested some performance improvements that are worth addressing. * Inline `IntPtr` constants for `LCompare`
André suggested some performance improvements that are worth addressing. * Inline `IntPtr` constants for `LCompare` * Avoid two `MArrayBufferViewLength` for `.length` (Int32) and bounds check (IntPtr) with a separate IntPtrToInt32 guard * Related, optimize bounds check hoisting better for trivial `i < ta.length` case * See if we can use redefine for no-op `MInt32ToIntPtr`
André suggested some performance improvements that are worth addressing. * Inline `IntPtr` constants for `LCompare` * Avoid two `MArrayBufferViewLength` for `.length` (Int32) and bounds check (IntPtr) with a separate IntPtrToInt32 guard * Related, optimize bounds check hoisting better for trivial `i < ta.length` case * See if we can use redefine for no-op `MInt32ToIntPtr` * `move64` optimization: https://bugzilla.mozilla.org/show_bug.cgi?id=1586662 * Fix `AnalyzeLoadUnboxedScalar`, although it has no effect with Spectre mitigations on
André suggested some performance improvements that are worth addressing. * Inline `IntPtr` constants for `LCompare` * Avoid two `MArrayBufferViewLength` for `.length` (Int32) and bounds check (IntPtr) with a separate IntPtrToInt32 guard * Related, optimize bounds check hoisting better for trivial `i < ta.length` case * See if we can use redefine for no-op `MInt32ToIntPtr` * `move64` optimization: https://bugzilla.mozilla.org/show_bug.cgi?id=1586662 * Fix `AnalyzeLoadUnboxedScalar`, although it has no effect with Spectre mitigations on * Support `IntPtr` (for `MInt32ToIntPtr`) in `RangeAnalysis::addRangeAssertions`.
André suggested some performance improvements that are worth addressing. * Inline `IntPtr` constants for `LCompare` * Avoid two `MArrayBufferViewLength` for `.length` (Int32) and bounds check (IntPtr) with a separate IntPtrToInt32 guard * Related, optimize bounds check hoisting better for trivial `i < ta.length` case * See if we can use redefine for no-op `MInt32ToIntPtr` * `move64` optimization: https://bugzilla.mozilla.org/show_bug.cgi?id=1586662 * Fix `AnalyzeLoadUnboxedScalar`, although it has no effect with Spectre mitigations on * Support `IntPtr` (for `MInt32ToIntPtr`) in `RangeAnalysis::addRangeAssertions`. * Propagate range for `MNonNegativeIntPtrToInt32`.