[PATCH] D88321: [SVE][CodeGen] Lower scalable fp_extend & fp_round operations

Paul Walker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 29 02:46:51 PDT 2020


paulwalker-arm accepted this revision.
paulwalker-arm added a comment.
This revision is now accepted and ready to land.

FP_ROUND's extra operand is a tad inconvenient but that's not your fault and you're following the naming rules for _MERGE_PASSTHRU so this LGTM. I'll note there's no illegal to illegal test for fptrunc like there is for fpext but then I don't know what those tests are verifying that's not already covered by the illegal<->legal tests.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D88321/new/

https://reviews.llvm.org/D88321



More information about the llvm-commits mailing list