Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(461)

Unified Diff: src/compiler/store-store-elimination.cc

Issue 2452403003: Changed statement ZoneList to a ZoneChunkList
Patch Set: Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « src/compiler/state-values-utils.cc ('k') | src/crankshaft/arm/lithium-arm.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/compiler/store-store-elimination.cc
diff --git a/src/compiler/store-store-elimination.cc b/src/compiler/store-store-elimination.cc
index 196cb0d60819e861c1dd5a0744b416007a8d5124..0ec9d98eff788ac7e61dc4c372e99ac47ec7582d 100644
--- a/src/compiler/store-store-elimination.cc
+++ b/src/compiler/store-store-elimination.cc
@@ -473,8 +473,7 @@ UnobservablesSet UnobservablesSet::VisitedEmpty(Zone* zone) {
// Create a new empty UnobservablesSet. This allocates in the zone, and
// can probably be optimized to use a global singleton.
ZoneSet<UnobservableStore>* empty_set =
- new (zone->New(sizeof(ZoneSet<UnobservableStore>)))
- ZoneSet<UnobservableStore>(zone);
+ new (zone) ZoneSet<UnobservableStore>(zone);
return UnobservablesSet(empty_set);
}
@@ -487,8 +486,7 @@ UnobservablesSet UnobservablesSet::Intersect(UnobservablesSet other,
return Unvisited();
} else {
ZoneSet<UnobservableStore>* intersection =
- new (zone->New(sizeof(ZoneSet<UnobservableStore>)))
- ZoneSet<UnobservableStore>(zone);
+ new (zone) ZoneSet<UnobservableStore>(zone);
// Put the intersection of set() and other.set() in intersection.
set_intersection(set()->begin(), set()->end(), other.set()->begin(),
other.set()->end(),
@@ -506,8 +504,7 @@ UnobservablesSet UnobservablesSet::Add(UnobservableStore obs,
} else {
// Make a new empty set.
ZoneSet<UnobservableStore>* new_set =
- new (zone->New(sizeof(ZoneSet<UnobservableStore>)))
- ZoneSet<UnobservableStore>(zone);
+ new (zone) ZoneSet<UnobservableStore>(zone);
// Copy the old elements over.
*new_set = *set();
// Add the new element.
@@ -523,8 +520,7 @@ UnobservablesSet UnobservablesSet::RemoveSameOffset(StoreOffset offset,
Zone* zone) const {
// Make a new empty set.
ZoneSet<UnobservableStore>* new_set =
- new (zone->New(sizeof(ZoneSet<UnobservableStore>)))
- ZoneSet<UnobservableStore>(zone);
+ new (zone) ZoneSet<UnobservableStore>(zone);
// Copy all elements over that have a different offset.
for (auto obs : *set()) {
if (obs.offset_ != offset) {
« no previous file with comments | « src/compiler/state-values-utils.cc ('k') | src/crankshaft/arm/lithium-arm.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698