AuroraMiddleware/v8
1
0
Fork 0
Code Issues 2 Pull Requests Releases Wiki Activity

X87: Get rid of AllocationFlags::TAG_OBJECT.

Browse Source 
port ef49c6b11a (r35792)

  original commit message:
  Default (and only way) is now to retrieve a tagged object.

BUG=

Review URL: https://codereview.chromium.org/1924643004

Cr-Commit-Position: refs/heads/master@{#35808}
This commit is contained in:
zhengxing.li 2016-04-27 00:30:24 -07:00 committed by Commit bot
parent 97c357aac1
commit 52e946ea90
2 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/x87/codegen-x87.cc
View File

@ -275,9 +275,9 @@ void ElementsTransitionGenerator::GenerateSmiToDouble(
// Allocate new FixedDoubleArray.
// edx: receiver
// edi: length of source FixedArray (smi-tagged)
AllocationFlags flags = static_cast<AllocationFlags>(DOUBLE_ALIGNMENT);
__ Allocate(FixedDoubleArray::kHeaderSize, times_8, edi,
REGISTER_VALUE_IS_SMI, eax, ebx, no_reg, &gc_required,
DOUBLE_ALIGNMENT);
REGISTER_VALUE_IS_SMI, eax, ebx, no_reg, &gc_required, flags);
// eax: destination FixedDoubleArray
// edi: number of elements

3
src/x87/macro-assembler-x87.cc
View File

@ -1520,10 +1520,10 @@ void MacroAssembler::Allocate(int object_size,
// Update allocation top.
UpdateAllocationTopHelper(top_reg, scratch, flags);
// Tag result if requested.
if (top_reg.is(result)) {
sub(result, Immediate(object_size - kHeapObjectTag));
} else {
// Tag the result.
DCHECK(kHeapObjectTag == 1);
inc(result);
}
@ -1597,6 +1597,7 @@ void MacroAssembler::Allocate(int header_size,
cmp(result_end, Operand::StaticVariable(allocation_limit));
j(above, gc_required);
// Tag result.
DCHECK(kHeapObjectTag == 1);
inc(result);