Page MenuHomePhabricator

[TTI] Account for FMA opportunities in getArithmeticInstrCost.
Needs ReviewPublic

Authored by fhahn on Aug 30 2022, 12:42 PM.

Details

Summary

TTI-based alternative to D132872.

Depends on D132967.

Diff Detail

Event Timeline

fhahn created this revision.Aug 30 2022, 12:42 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 30 2022, 12:42 PM
Herald added a subscriber: hiraditya. · View Herald Transcript
fhahn requested review of this revision.Aug 30 2022, 12:42 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 30 2022, 12:42 PM
ABataev added inline comments.Aug 30 2022, 1:15 PM
llvm/lib/Analysis/TargetTransformInfo.cpp
803

Use TTI::TCC_Free

llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp
6470

== TTI::TCC_Free

fhahn updated this revision to Diff 456977.Aug 31 2022, 8:33 AM

Address latest comments, also add early exit to lambda to reduce indent, compare against FAdd cost as suggested by @dmgreen.

fhahn marked 2 inline comments as done.Aug 31 2022, 8:40 AM
fhahn added inline comments.
llvm/lib/Analysis/TargetTransformInfo.cpp
803

Fixed, thanks!

llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp
6470

Fixed, thanks!