- Refactor for costs of sqrt/fabs
 - Add half type support for the cost model of sqrt/fabs
 
Details
Details
Diff Detail
Diff Detail
- Repository
 - rG LLVM Github Monorepo
 
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 | ||
|---|---|---|
| 836 | 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 | ||
|---|---|---|
| 852 | 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.