[heap] Remove retry space from AllocateRaw.

[heap] Remove retry space from AllocateRaw.

BUG=

Committed: https://crrev.com/2786ceec1eb491494d24d29a59eefbe3b6704be2
Cr-Commit-Position: refs/heads/master@{#30984}

[Hannes Payer (out of office), 2015-09-28 15:12:08]

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

[Michael Starzinger, 2015-09-28 16:07:50]

LGTM with one comment to address.

https://codereview.chromium.org/1370123002/diff/20001/src/heap/heap-inl.h
File src/heap/heap-inl.h (right):

https://codereview.chromium.org/1370123002/diff/20001/src/heap/heap-inl.h#new...
src/heap/heap-inl.h:567: __allocation__ = FUNCTION_CALL;                        
                  \
As discussed offline: I think we should still keep the AlwaysAllocateScope here,
because the GCs above might not have been able to reclaim memory and we want
"maximum expansion criteria" for this last attempt in old spaces. Basically I
propose to leave CALL_HEAP_FUNCTION unchanged with this CL.

[Hannes Payer (out of office), 2015-09-28 16:11:49]

https://codereview.chromium.org/1370123002/diff/20001/src/heap/heap-inl.h
File src/heap/heap-inl.h (right):

https://codereview.chromium.org/1370123002/diff/20001/src/heap/heap-inl.h#new...
src/heap/heap-inl.h:567: __allocation__ = FUNCTION_CALL;                        
                  \
On 2015/09/28 16:07:50, Michael Starzinger wrote:
> As discussed offline: I think we should still keep the AlwaysAllocateScope
here,
> because the GCs above might not have been able to reclaim memory and we want
> "maximum expansion criteria" for this last attempt in old spaces. Basically I
> propose to leave CALL_HEAP_FUNCTION unchanged with this CL.

Done.

[Hannes Payer (out of office), 2015-09-28 16:11:52]

The CQ bit was checked by hpayer@chromium.org

[Hannes Payer (out of office), 2015-09-28 16:11:52]

The patchset sent to the CQ was uploaded after l-g-t-m from
mstarzinger@chromium.org
Link to the patchset: https://codereview.chromium.org/1370123002/#ps40001
(title: " ")

[commit-bot: I haz the power, 2015-09-28 16:12:08]

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

[commit-bot: I haz the power, 2015-09-28 18:07:13]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-09-28 18:07:40]

Patchset 3 (id:??) landed as
https://crrev.com/2786ceec1eb491494d24d29a59eefbe3b6704be2
Cr-Commit-Position: refs/heads/master@{#30984}

[Benedikt Meurer, 2015-09-29 04:39:33]

A revert of this CL (patchset #3 id:40001) has been created in
https://codereview.chromium.org/1379493003/ by bmeurer@chromium.org.

The reason for reverting is: Breaks arm64 nosnap simulator (bogus cctest?):
https://chromegw.corp.google.com/i/client.v8/builders/V8%20Linux%20-%20arm64%....