Page MenuHomePhabricator
Differential D109678

[LV][NFC] Cache scalar cost.
Needs ReviewPublic
Actions

Authored by ebrevnov on Sep 13 2021, 3:01 AM.

Details

Reviewers
fhahn
Ayal
anna
lebedev.ri
Summary

There is a need to know scalar cost in different places of cost model (see dependences). Cache it as a private member of the cost model instance.

Diff Detail

Event Timeline

ebrevnov created this revision.Sep 13 2021, 3:01 AM
Herald added a subscriber: hiraditya. · View Herald TranscriptSep 13 2021, 3:01 AM
ebrevnov requested review of this revision.Sep 13 2021, 3:01 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 13 2021, 3:01 AM
Herald added a subscriber: llvm-commits. · View Herald Transcript
ebrevnov added a parent revision: D109677: [LV][NFC] Add 'const' modifier to getInstructionCost* members.Sep 13 2021, 3:08 AM
ebrevnov added a child revision: D109679: [LV][NFC] Optimize out an extra call to isMoreProfitable.
Harbormaster completed remote builds in B123637: Diff 372196.Sep 13 2021, 3:11 AM
ebrevnov edited the summary of this revision. (Show Details)Sep 13 2021, 5:21 AM
ebrevnov added reviewers: fhahn, lebedev.ri, Ayal, anna.
ebrevnov mentioned this in D75981: [LV] Create RT checks once VF/IC are selected, track scalar cost..Oct 1 2021, 12:14 AM
lebedev.ri resigned from this revision.Jan 12 2023, 5:24 PM

This review seems to be stuck/dead, consider abandoning if no longer relevant.

Herald added a project: Restricted Project. · View Herald TranscriptJan 12 2023, 5:24 PM
Herald added subscribers: pcwang-thead, StephenFan. · View Herald Transcript

Revision Contents
Changeset List

PathSize
llvm/
lib/
Transforms/
Vectorize/
29 lines

Diff 372196

View Options

llvm/lib/Transforms/Vectorize/LoopVectorize.cpp

Loading...