Revision: 16980
Author: [email protected]
Date: Fri Sep 27 09:50:47 2013 UTC
Log: Merged r16355 into 3.20 branch.
Filler sizes have to be Smis, fix x64 breakage.
[email protected]
Review URL: https://codereview.chromium.org/25011002
http://code.google.com/p/v8/source/detail?r=16980
Modified:
/branches/3.20/src/hydrogen-instructions.cc
/branches/3.20/src/hydrogen-instructions.h
/branches/3.20/src/version.cc
=======================================
--- /branches/3.20/src/hydrogen-instructions.cc Thu Sep 26 13:29:44 2013 UTC
+++ /branches/3.20/src/hydrogen-instructions.cc Fri Sep 27 09:50:47 2013 UTC
@@ -3268,7 +3268,11 @@
}
HInstruction* new_dominator_size_constant =
HConstant::CreateAndInsertBefore(
- zone, context(), new_dominator_size, dominator_allocate);
+ zone,
+ context(),
+ new_dominator_size,
+ Representation::None(),
+ dominator_allocate);
dominator_allocate->UpdateSize(new_dominator_size_constant);
#ifdef VERIFY_HEAP
@@ -3371,11 +3375,15 @@
void HAllocate::UpdateFreeSpaceFiller(int32_t free_space_size) {
ASSERT(filler_free_space_size_ != NULL);
Zone* zone = block()->zone();
+ // We must explicitly force Smi representation here because on x64 we
+ // would otherwise automatically choose int32, but the actual store
+ // requires a Smi-tagged value.
HConstant* new_free_space_size = HConstant::CreateAndInsertBefore(
zone,
context(),
filler_free_space_size_->value()->GetInteger32Constant() +
free_space_size,
+ Representation::Smi(),
filler_free_space_size_);
filler_free_space_size_->UpdateValue(new_free_space_size);
}
@@ -3401,10 +3409,15 @@
store_map->SetFlag(HValue::kHasNoObservableSideEffects);
store_map->InsertAfter(filler_map);
+ // We must explicitly force Smi representation here because on x64 we
+ // would otherwise automatically choose int32, but the actual store
+ // requires a Smi-tagged value.
HConstant* filler_size = HConstant::CreateAndInsertAfter(
- zone, context(), free_space_size, store_map);
+ zone, context(), free_space_size, Representation::Smi(), store_map);
+ // Must force Smi representation for x64 (see comment above).
HObjectAccess access =
- HObjectAccess::ForJSObjectOffset(FreeSpace::kSizeOffset);
+ HObjectAccess::ForJSObjectOffset(FreeSpace::kSizeOffset,
+ Representation::Smi());
HStoreNamedField* store_size = HStoreNamedField::New(zone, context(),
free_space_instr, access, filler_size);
store_size->SetFlag(HValue::kHasNoObservableSideEffects);
=======================================
--- /branches/3.20/src/hydrogen-instructions.h Thu Sep 26 13:29:44 2013 UTC
+++ /branches/3.20/src/hydrogen-instructions.h Fri Sep 27 09:50:47 2013 UTC
@@ -3210,8 +3210,10 @@
static HConstant* CreateAndInsertAfter(Zone* zone,
HValue* context,
int32_t value,
+ Representation representation,
HInstruction* instruction) {
- HConstant* new_constant = HConstant::New(zone, context, value);
+ HConstant* new_constant =
+ HConstant::New(zone, context, value, representation);
new_constant->InsertAfter(instruction);
return new_constant;
}
@@ -3219,8 +3221,10 @@
static HConstant* CreateAndInsertBefore(Zone* zone,
HValue* context,
int32_t value,
+ Representation representation,
HInstruction* instruction) {
- HConstant* new_constant = HConstant::New(zone, context, value);
+ HConstant* new_constant =
+ HConstant::New(zone, context, value, representation);
new_constant->InsertBefore(instruction);
return new_constant;
}
=======================================
--- /branches/3.20/src/version.cc Thu Sep 26 13:29:44 2013 UTC
+++ /branches/3.20/src/version.cc Fri Sep 27 09:50:47 2013 UTC
@@ -35,7 +35,7 @@
#define MAJOR_VERSION 3
#define MINOR_VERSION 20
#define BUILD_NUMBER 17
-#define PATCH_LEVEL 13
+#define PATCH_LEVEL 14
// Use 1 for candidates and 0 otherwise.
// (Boolean macro values are not supported by all preprocessors.)
#define IS_CANDIDATE_VERSION 0
--
--
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/groups/opt_out.
