Title: [193983] trunk/Source/_javascript_Core
- Revision
- 193983
- Author
- [email protected]
- Date
- 2015-12-11 14:39:29 -0800 (Fri, 11 Dec 2015)
Log Message
Overrides has instance should not move ValueFalse to a register then immediately to the stack in the LLInt.
https://bugs.webkit.org/show_bug.cgi?id=152188
Reviewed by Mark Lam.
This fixes a minor issue with the code for the overrides_has_instance in the LLInt. Old code had an extra move,
which is both slow and breaks the build on cloop.
* llint/LowLevelInterpreter64.asm:
Modified Paths
Diff
Modified: trunk/Source/_javascript_Core/ChangeLog (193982 => 193983)
--- trunk/Source/_javascript_Core/ChangeLog 2015-12-11 22:33:40 UTC (rev 193982)
+++ trunk/Source/_javascript_Core/ChangeLog 2015-12-11 22:39:29 UTC (rev 193983)
@@ -1,5 +1,17 @@
2015-12-11 Keith Miller <[email protected]>
+ Overrides has instance should not move ValueFalse to a register then immediately to the stack in the LLInt.
+ https://bugs.webkit.org/show_bug.cgi?id=152188
+
+ Reviewed by Mark Lam.
+
+ This fixes a minor issue with the code for the overrides_has_instance in the LLInt. Old code had an extra move,
+ which is both slow and breaks the build on cloop.
+
+ * llint/LowLevelInterpreter64.asm:
+
+2015-12-11 Keith Miller <[email protected]>
+
[ES6] Add support for Symbol.hasInstance
https://bugs.webkit.org/show_bug.cgi?id=151839
Modified: trunk/Source/_javascript_Core/llint/LowLevelInterpreter64.asm (193982 => 193983)
--- trunk/Source/_javascript_Core/llint/LowLevelInterpreter64.asm 2015-12-11 22:33:40 UTC (rev 193982)
+++ trunk/Source/_javascript_Core/llint/LowLevelInterpreter64.asm 2015-12-11 22:39:29 UTC (rev 193983)
@@ -1088,8 +1088,7 @@
dispatch(4)
.opOverridesHasInstanceNotDefaultSymbol:
- storeq ValueTrue, t1
- storeq t1, [cfr, t3, 8]
+ storeq ValueTrue, [cfr, t3, 8]
dispatch(4)
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
