1214562 - Cleanup SetPropertyCache/SetElementCache regalloc

Status: RESOLVED FIXED

(Core :: JavaScript Engine: JIT, defect)

Description

[Jan de Mooij [:jandem]]

Unifying them is a bit complicated due to their current regalloc contraints. I'll post some patches to clean this up.

Comment 1

[Jan de Mooij [:jandem]]

Attachment: Part 1 - Don't use useByteOpRegister

LSetElementCacheV uses useByteOpRegister becaues that was needed for the typed array stub.

You fixed the x86 assembler to handle this more transparently so we no longer need this.

Comment 2

[Jan de Mooij [:jandem]]

Attachment: Part 2 - Refactor SetPropertyCache

This patch:

* Uses useBoxOrTypedOrConstant, so we can use a single LIR instruction for SetPropertyCache.

* Gives SetPropertyCache always a temp register instead of using a tempCopy. This will make bug 1214126 a lot nicer and simplifies many stubs because we need fewer stack pushes/pops.

Comment 3

[Brian Hackett [Laid off!]]

Comment on attachment 8673589 [details] [diff] [review]
Part 1 - Don't use useByteOpRegister

Review of attachment 8673589 [details] [diff] [review]:
-----------------------------------------------------------------

::: js/src/jit/Lowering.cpp
@@ +3527,5 @@
>  
>      // Due to lack of registers on x86, we reuse the object register as a
>      // temporary. This register may be used in a 1-byte store, which on x86
>      // again has constraints; thus the use of |useByteOpRegister| over
>      // |useRegister| below.

This comment should be fixed.

Comment 4

[Brian Hackett [Laid off!]]

Comment on attachment 8673598 [details] [diff] [review]
Part 2 - Refactor SetPropertyCache

Review of attachment 8673598 [details] [diff] [review]:
-----------------------------------------------------------------

Nice!

::: js/src/jit/IonCaches.cpp
@@ +2239,5 @@
>      lastJump_ = initialJump_;
>  }
>  
>  // Jump to failure if a value being written is not a property for obj/id.
>  // This might clobber |object|.

This comment should be fixed.

Comment 5

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/0ea76f0263ef

Comment 6

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/73e1760e3c4d

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/0ea76f0263ef
https://hg.mozilla.org/mozilla-central/rev/73e1760e3c4d