Following on from D102353, extend the fpto*i.sat intrinsics to use NEON
fcvt* instructions.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
llvm/lib/Target/AArch64/AArch64TargetTransformInfo.cpp | ||
---|---|---|
991 ↗ | (On Diff #367770) | I omitted this suggestion, applying the golden rule. The 450 lines of diff made the actual change non-obvious (with default diff options), but I can include it if that's preferred. |
llvm/lib/Target/AArch64/AArch64ISelLowering.cpp | ||
---|---|---|
3396 | I don't think this "Warning" adds much. It's OK to remove for now. | |
llvm/lib/Target/AArch64/AArch64TargetTransformInfo.cpp | ||
1084 ↗ | (On Diff #367770) | This would need tests, and is probably best left for a separate commit. I think there would need to be more general FP_TO_SINT_SAT costing changes, it won't ever get here at the moment as they are intrinsics not cast instructions. |
llvm/test/CodeGen/AArch64/fptosi-sat-vector.ll | ||
2–3 | Can you add a run line with fp16 too. |
I don't think this "Warning" adds much. It's OK to remove for now.