More cleanup regarding the maximum non-large object allocation size.

More cleanup regarding the maximum non-large object allocation size.

BUG=
R=mstarzinger@chromium.org

Committed: https://code.google.com/p/v8/source/detail?r=16052

[Michael Starzinger, 2013-08-02 12:30:53]

Looking good, just one comment.

https://codereview.chromium.org/20867003/diff/3001/src/spaces.h
File src/spaces.h (right):

https://codereview.chromium.org/20867003/diff/3001/src/spaces.h#newcode783
src/spaces.h:783: kPageSize - kObjectStartOffset - kPointerSize;
I don't think the wiggle room for double alignment should be taken into account
for the "area size", IMHO it should only apply to the "max object size" a lines
below.

[Michael Starzinger, 2013-08-02 12:31:17]

LGTM after previous comment is addressed.

[Hannes Payer (out of office), 2013-08-05 12:52:21]

https://codereview.chromium.org/20867003/diff/3001/src/spaces.h
File src/spaces.h (right):

https://codereview.chromium.org/20867003/diff/3001/src/spaces.h#newcode783
src/spaces.h:783: kPageSize - kObjectStartOffset - kPointerSize;
On 2013/08/02 12:30:53, Michael Starzinger wrote:
> I don't think the wiggle room for double alignment should be taken into
account
> for the "area size", IMHO it should only apply to the "max object size" a
lines
> below.

Done.

[Hannes Payer (out of office), 2013-08-05 12:53:00]

Committed patchset #4 manually as r16052 (presubmit successful).