PPC: [Heap] Fix comparing against new space top pointer

PPC: [Heap] Fix comparing against new space top pointer

Port d6473f5c14de5f52866b41730807932158fcaffb

Original commit message:

    See bug description.

R=mlippautz@chromium.org, joransiu@ca.ibm.com, jyan@ca.ibm.com, michael_dawson@ca.ibm.com, mbrandy@us.ibm.com

BUG=chromium:619382
LOG=N

Committed: https://crrev.com/b95de0441228b874bd125e9afe18e08b5b02b918
Cr-Commit-Position: refs/heads/master@{#37023}

[JaideepBajwa, 2016-06-16 05:05:23]

PTAL

[john.yan, 2016-06-16 05:13:49]

just one comment otherwise lgtm

https://codereview.chromium.org/2066603007/diff/1/src/ppc/macro-assembler-ppc.cc
File src/ppc/macro-assembler-ppc.cc (right):

https://codereview.chromium.org/2066603007/diff/1/src/ppc/macro-assembler-ppc...
src/ppc/macro-assembler-ppc.cc:4734: mov(ip,
Operand(new_space_allocation_top_adr));
I am not sure why we need to do the load again.

[JaideepBajwa, 2016-06-16 05:26:02]

On 2016/06/16 05:13:49, john.yan wrote:
> just one comment otherwise lgtm
> 
>
https://codereview.chromium.org/2066603007/diff/1/src/ppc/macro-assembler-ppc.cc
> File src/ppc/macro-assembler-ppc.cc (right):
> 
>
https://codereview.chromium.org/2066603007/diff/1/src/ppc/macro-assembler-ppc...
> src/ppc/macro-assembler-ppc.cc:4734: mov(ip,
> Operand(new_space_allocation_top_adr));
> I am not sure why we need to do the load again.

Yes ip doesn't seem to be used between the 2 loads. I'll add the fix.
Thanks.

[JaideepBajwa, 2016-06-16 05:59:52]

The CQ bit was checked by bjaideep@ca.ibm.com

[JaideepBajwa, 2016-06-16 05:59:53]

The patchset sent to the CQ was uploaded after l-g-t-m from jyan@ca.ibm.com
Link to the patchset: https://codereview.chromium.org/2066603007/#ps20001
(title: "addressed comment, removed redundant load")

[commit-bot: I haz the power, 2016-06-16 06:00:03]

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

[commit-bot: I haz the power, 2016-06-16 06:25:44]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-16 06:26:30]

Description was changed from

==========
PPC: [Heap] Fix comparing against new space top pointer

Port d6473f5c14de5f52866b41730807932158fcaffb

Original commit message:

    See bug description.

R=mlippautz@chromium.org, joransiu@ca.ibm.com, jyan@ca.ibm.com,
michael_dawson@ca.ibm.com, mbrandy@us.ibm.com

BUG=chromium:619382
LOG=N
==========

to

==========
PPC: [Heap] Fix comparing against new space top pointer

Port d6473f5c14de5f52866b41730807932158fcaffb

Original commit message:

    See bug description.

R=mlippautz@chromium.org, joransiu@ca.ibm.com, jyan@ca.ibm.com,
michael_dawson@ca.ibm.com, mbrandy@us.ibm.com

BUG=chromium:619382
LOG=N

Committed: https://crrev.com/b95de0441228b874bd125e9afe18e08b5b02b918
Cr-Commit-Position: refs/heads/master@{#37023}
==========

[commit-bot: I haz the power, 2016-06-16 06:26:31]

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