Reviewers: danno, Michael Starzinger, paul.l..., dusmil.imgtec,
akos.palfi.imgtec,
Description:
MIPS64: Refine 'Remove unused byte from Map::instance_sizes field.'
Port 2027335f1ce44b0c6523eb5e55881e2a4e162815
Original commit message:
Note that there are currently no objects that require a pre-allocated
properties backing store, all such slots are in-object properties from
the begining. Hence {unused + pre_allocated - inobject == 0} holds.
BUG=
Please review this at https://codereview.chromium.org/1226363003/
Base URL: https://chromium.googlesource.com/v8/v8.git@master
Affected files (+2, -6 lines):
M src/mips64/builtins-mips64.cc
Index: src/mips64/builtins-mips64.cc
diff --git a/src/mips64/builtins-mips64.cc b/src/mips64/builtins-mips64.cc
index
6305a79203125435eb88256ddab7039ed127c874..0300dc8b9a21ef36a8803b2e0106d143f6fe023e
100644
--- a/src/mips64/builtins-mips64.cc
+++ b/src/mips64/builtins-mips64.cc
@@ -483,12 +483,8 @@ static void
Generate_JSConstructStubHelper(MacroAssembler* masm,
Operand(static_cast<int64_t>(Map::kSlackTrackingCounterEnd)));
// Allocate object with a slack.
- __ lwu(a0, FieldMemOperand(a2, Map::kInstanceSizesOffset));
- __ Ext(a0, a0, Map::kInObjectPropertiesByte * kBitsPerByte,
- kBitsPerByte);
- __ lwu(a2, FieldMemOperand(a2, Map::kInstanceAttributesOffset));
- __ Ext(a2, a2, Map::kUnusedPropertyFieldsByte * kBitsPerByte,
- kBitsPerByte);
+ __ lbu(a0, FieldMemOperand(a2, Map::kInObjectPropertiesOffset));
+ __ lbu(a2, FieldMemOperand(a2, Map::kUnusedPropertyFieldsOffset));
__ dsubu(a0, a0, a2);
__ dsll(at, a0, kPointerSizeLog2);
__ daddu(a0, t1, at);
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
