[x64] Use movaps instead of movsd for the move between double registers.

[x64] Use movaps instead of movsd for the move between double registers.

Committed: https://crrev.com/11d97bf531676392fee2cb66eaf0b15f8d423e3d
Cr-Commit-Position: refs/heads/master@{#26950}

[Weiliang, 2015-03-03 05:50:40]

weiliang.lin@intel.com changed reviewers:
+ bmeurer@chromium.org, danno@chromium.org, dcarney@chromium.org,
titzer@chromium.org

[Weiliang, 2015-03-03 05:50:41]

PTAL

[Benedikt Meurer, 2015-03-03 05:54:34]

A few years ago I did a similar patch for OCaml
(http://caml.inria.fr/mantis/view.php?id=5180), but using movapd, which gave a
nice speed up then. I remember that I tried to change this in V8 once, but it
didn't make any difference on our Xeon's.

I'm fine with landing this CL, just two questions:

(a) Should we use movapd instead of movaps?
(b) Should we do this all over the place on x64, not just in TurboFan?

[Weiliang, 2015-03-03 06:12:55]

On 2015/03/03 05:54:34, Benedikt Meurer wrote:
> A few years ago I did a similar patch for OCaml
> (http://caml.inria.fr/mantis/view.php?id=5180), but using movapd, which gave a
> nice speed up then. I remember that I tried to change this in V8 once, but it
> didn't make any difference on our Xeon's.
> 
> I'm fine with landing this CL, just two questions:
> 
> (a) Should we use movapd instead of movaps?
> (b) Should we do this all over the place on x64, not just in TurboFan?

The patch is just to align with Crankshaft, which has already use movaps in x64
backend(src/x64/lithium-gap-resolver-x64.cc).
It also restores the regression caused by r26849 on HSW when AVX is disabled.

Thanks
-Weiliang

[Benedikt Meurer, 2015-03-03 06:13:23]

lgtm

[Benedikt Meurer, 2015-03-03 06:13:40]

The CQ bit was checked by bmeurer@chromium.org

[commit-bot: I haz the power, 2015-03-03 06:15:01]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/972053002/1

[commit-bot: I haz the power, 2015-03-03 06:34:19]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-03-03 06:34:36]

Patchset 1 (id:??) landed as
https://crrev.com/11d97bf531676392fee2cb66eaf0b15f8d423e3d
Cr-Commit-Position: refs/heads/master@{#26950}