Commit cf804f26 authored by Tobias Tebbi's avatar Tobias Tebbi Committed by V8 LUCI CQ

[compiler] only emit constant loads for constant owner map

Bug: chromium:1323114
Change-Id: I2ec6c7d5eb5ef08914cfc1fac52d0128ecb27c53
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3630081Reviewed-by: 's avatarNico Hartmann <nicohartmann@chromium.org>
Commit-Queue: Nico Hartmann <nicohartmann@chromium.org>
Auto-Submit: Tobias Tebbi <tebbi@chromium.org>
Cr-Commit-Position: refs/heads/main@{#80417}
parent b8b7a3a2
......@@ -482,12 +482,8 @@ PropertyAccessInfo AccessInfoFactory::ComputeDataFieldAccessInfo(
dependencies()->FieldTypeDependencyOffTheRecord(
map, descriptor, descriptors_field_type_ref.value()));
PropertyConstness constness;
if (details.IsReadOnly() && !details.IsConfigurable()) {
constness = PropertyConstness::kConst;
} else {
constness = dependencies()->DependOnFieldConstness(map, descriptor);
}
PropertyConstness constness =
dependencies()->DependOnFieldConstness(map, descriptor);
// Note: FindFieldOwner may be called multiple times throughout one
// compilation. This is safe since its result is fixed for a given map and
......
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