This reuses the getArithmeticInstrCost, but passes dummy values of the second
operand flags.
The X86 costs are wrong and can be improved in a follow up. I just wanted to
stop it from reporting an unknown cost first.
Paths
| Differential D62444
[CostModel] Add really basic support for being able to query the cost of the FNeg instruction. ClosedPublic Authored by craig.topper on May 24 2019, 10:55 PM.
Details Summary This reuses the getArithmeticInstrCost, but passes dummy values of the second The X86 costs are wrong and can be improved in a follow up. I just wanted to
Diff Detail
Event TimelineComment Actions Add the tests first as a preliminary step? Comment Actions
Yeah doesn't look like we recognize fsub -0.0, X as special currently. Comment Actions Really rebase it. I hadn't pulled after pushing the test so arcanist merged the two commits. This revision is now accepted and ready to land.May 27 2019, 7:24 AM Closed by commit rL361788: [CostModel] Add really basic support for being able to query the cost of the… (authored by ctopper). · Explain WhyMay 27 2019, 9:08 PM This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 201606 llvm/trunk/lib/Analysis/TargetTransformInfo.cpp
llvm/trunk/test/Analysis/CostModel/X86/arith-fp.ll
|