Title: [231044] trunk/Source/_javascript_Core
- Revision
- 231044
- Author
- [email protected]
- Date
- 2018-04-26 05:04:22 -0700 (Thu, 26 Apr 2018)
Log Message
[MIPS] Fix branch offsets in branchNeg32
https://bugs.webkit.org/show_bug.cgi?id=185025
Patch by Dominik Infuehr <[email protected]> on 2018-04-26
Reviewed by Yusuke Suzuki.
Two nops were removed in branch(Not)Equal in #183130 but the offset wasn't adjusted.
* assembler/MacroAssemblerMIPS.h:
(JSC::MacroAssemblerMIPS::branchNeg32):
Modified Paths
Diff
Modified: trunk/Source/_javascript_Core/ChangeLog (231043 => 231044)
--- trunk/Source/_javascript_Core/ChangeLog 2018-04-26 07:54:19 UTC (rev 231043)
+++ trunk/Source/_javascript_Core/ChangeLog 2018-04-26 12:04:22 UTC (rev 231044)
@@ -1,3 +1,15 @@
+2018-04-26 Dominik Infuehr <[email protected]>
+
+ [MIPS] Fix branch offsets in branchNeg32
+ https://bugs.webkit.org/show_bug.cgi?id=185025
+
+ Reviewed by Yusuke Suzuki.
+
+ Two nops were removed in branch(Not)Equal in #183130 but the offset wasn't adjusted.
+
+ * assembler/MacroAssemblerMIPS.h:
+ (JSC::MacroAssemblerMIPS::branchNeg32):
+
2018-04-25 Robin Morisset <[email protected]>
In FTLLowerDFGToB3.cpp::compileCreateRest, always use a contiguous array as the indexing type when under isWatchingHavingABadTimeWatchpoint
Modified: trunk/Source/_javascript_Core/assembler/MacroAssemblerMIPS.h (231043 => 231044)
--- trunk/Source/_javascript_Core/assembler/MacroAssemblerMIPS.h 2018-04-26 07:54:19 UTC (rev 231043)
+++ trunk/Source/_javascript_Core/assembler/MacroAssemblerMIPS.h 2018-04-26 12:04:22 UTC (rev 231044)
@@ -2457,9 +2457,9 @@
nop
No_overflow:
*/
- m_assembler.bgez(srcDest, 11);
+ m_assembler.bgez(srcDest, 9);
m_assembler.subu(srcDest, MIPSRegisters::zero, srcDest);
- m_assembler.bgez(srcDest, 9);
+ m_assembler.bgez(srcDest, 7);
m_assembler.nop();
return jump();
}
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
