- Refactor for costs of sqrt/fabs
- Add half type support for the cost model of sqrt/fabs
Details
Details
Diff Detail
Diff Detail
Event Timeline
Comment Actions
Please update the review description to describe actual change. In this case, I think you're adding half support.
llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp | ||
---|---|---|
681 | Unrelated change. Feel free to simply land a change which swaps the order here, but please remove it from this diff. |
llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp | ||
---|---|---|
704 | ST->hasVInstructionF16() is only true if hasVInstructions() is true so this || doesn't do anything. |
Unrelated change. Feel free to simply land a change which swaps the order here, but please remove it from this diff.