Commit d6b1f5bc authored by Georg Neis's avatar Georg Neis Committed by Commit Bot

[turbofan] Eliminate redundant TryUpdate call in access-info.cc

I had forgotten to remove this in
https://chromium-review.googlesource.com/c/v8/v8/+/1458243

Change-Id: Id0da5f87f782e854b204f400173aa2bbecb4c43c
Reviewed-on: https://chromium-review.googlesource.com/c/1470133Reviewed-by: 's avatarBenedikt Meurer <bmeurer@chromium.org>
Commit-Queue: Georg Neis <neis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#59578}
parent 685a20ca
......@@ -626,18 +626,10 @@ Maybe<ElementsKind> GeneralizeElementsKind(ElementsKind this_kind,
bool AccessInfoFactory::ConsolidateElementLoad(
MapHandles const& maps, ElementAccessInfo* access_info) const {
MapHandles updated_maps;
updated_maps.reserve(maps.size());
CHECK(!maps.empty());
InstanceType instance_type = maps.front()->instance_type();
ElementsKind elements_kind = maps.front()->elements_kind();
for (Handle<Map> map : maps) {
if (Map::TryUpdate(isolate(), map).ToHandle(&map)) {
updated_maps.push_back(map);
}
}
if (updated_maps.empty()) return false;
InstanceType instance_type = updated_maps.front()->instance_type();
ElementsKind elements_kind = updated_maps.front()->elements_kind();
for (Handle<Map> map : updated_maps) {
if (!CanInlineElementAccess(map) || map->instance_type() != instance_type) {
return false;
}
......@@ -646,7 +638,7 @@ bool AccessInfoFactory::ConsolidateElementLoad(
return false;
}
}
*access_info = ElementAccessInfo(updated_maps, elements_kind);
*access_info = ElementAccessInfo(maps, elements_kind);
return true;
}
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment