Closed Bug 1145811 Opened 9 years ago Closed 9 years ago

Clean-up: Remove simdSet argument of PushRegsInMask.

Categories

(Core :: JavaScript Engine: JIT, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla39
Tracking Status
firefox39 --- fixed
firefox40 --- fixed

People

(Reporter: nbp, Assigned: nbp)

References

Details

Attachments

(1 file)

      No description provided.
Noite: This patch is based on Bug 1143011 modififcations, which renames the
arguments to have a Live prefix to which indicate that the registers are
ideally the result of a register allocator.

Also, reduceSetForPush got also changed to effectively reduce the set of
register to only produce non-overlapping registers, as GetPushSizeInBytes
would return, at the same time where the iterator changed for non longer
iterating over Allocatable register, but over live registers.

The only major difference in terms of generated code when compared to the version
which was set before Bug 1143011 is that instead of doing a storeUnalignedInt32x4, we
are doing a storeUnalignedFloat32x4 for unknown SIMD registers.
Attachment #8580914 - Flags: review?(benj)
Comment on attachment 8580914 [details] [diff] [review]
Remove simdSet argument of PushRegsInMask.

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

Alright.

::: js/src/jit/shared/MacroAssembler-x86-shared.cpp
@@ +17,2 @@
>  {
> +    FloatRegisterSet doubleSet(set.fpus().reduceSetForPush());

nit: doubleSet isn't an appropriate name anymore. How about floatingPointSet or fpSet or fpuSet?

@@ +57,2 @@
>  {
> +    FloatRegisterSet doubleSet(set.fpus().reduceSetForPush());

ditto
Attachment #8580914 - Flags: review?(benj) → review+
Depends on: 1143011
https://hg.mozilla.org/mozilla-central/rev/8787eda5c93e
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla39
Depends on: 1149377
Re-landed on Aurora.
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: