[turbofan] Cleanup RawMachineAssembler::Store interface.

[turbofan] Cleanup RawMachineAssembler::Store interface.

R=bmeurer@chromium.org

Committed: https://crrev.com/26fc85aae3dedece5619f55ac4812578c25e0416
Cr-Commit-Position: refs/heads/master@{#31646}

[Michael Starzinger, 2015-10-28 14:14:46]

mstarzinger@chromium.org changed reviewers:
+ rmcilroy@chromium.org

[Michael Starzinger, 2015-10-28 14:14:47]

Benedikt: PTAL.
Ross: FYI.

https://codereview.chromium.org/1424983003/diff/1/src/compiler/raw-machine-as...
File src/compiler/raw-machine-assembler.h (right):

https://codereview.chromium.org/1424983003/diff/1/src/compiler/raw-machine-as...
src/compiler/raw-machine-assembler.h:504: return Store(rep,
PointerConstant(address), node, kNoWriteBarrier);
I think this default value is safe, because I cannot imagine a scenario where
storing to a "void*" pointer ever needs a write-barrier. Having a "void*"
pointing to a slot within the heap would be dangerous in itself already, the
write-barrier would be the last thing worrying me there. :)

Please let me know if you disagree.

[rmcilroy, 2015-10-28 15:57:46]

LGTM from my point of view.

[Benedikt Meurer, 2015-10-29 09:18:24]

LGTM

[Michael Starzinger, 2015-10-29 09:20:36]

The CQ bit was checked by mstarzinger@chromium.org

[commit-bot: I haz the power, 2015-10-29 09:20:44]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1424983003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1424983003/20001

[commit-bot: I haz the power, 2015-10-29 09:22:03]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-10-29 09:22:32]

Patchset 2 (id:??) landed as
https://crrev.com/26fc85aae3dedece5619f55ac4812578c25e0416
Cr-Commit-Position: refs/heads/master@{#31646}