Commit 208014ab authored by bmeurer's avatar bmeurer Committed by Commit bot

[turbofan] TurboFan doesn't use String allocation pretenuring feedback.

We can make Heap::UpdateAllocationSite cheaper for TurboFan, because
there's no need to check for allocation mementos on Strings in case of
the TurboFan pipeline.

R=mvstanton@chromium.org

Review-Url: https://codereview.chromium.org/2710303002
Cr-Commit-Position: refs/heads/master@{#43407}
parent 9db139e1
......@@ -1667,6 +1667,11 @@ AllocationSiteMode AllocationSite::GetMode(
}
inline bool AllocationSite::CanTrack(InstanceType type) {
if (FLAG_turbo) {
// TurboFan doesn't care at all about String pretenuring feedback,
// so don't bother even trying to track that.
return type == JS_ARRAY_TYPE || type == JS_OBJECT_TYPE;
}
if (FLAG_allocation_site_pretenuring) {
return type == JS_ARRAY_TYPE ||
type == JS_OBJECT_TYPE ||
......
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