[PATCH] D96037: [DAGCombiner] Don't fold FCOPYSIGN vector sign operand casts

Fraser Cormack via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 4 07:39:32 PST 2021


frasercrmck added a comment.

FWIW, something this would likely help with D96028 <https://reviews.llvm.org/D96028>, as RVV doesn't benefit from this fpextend/fptrunc folding. We have to undo it in the custom-lowering and use a custom-node to prevent infinite loops. I was wondering if the combiner should ask the target whether it's beneficial to fold the ext/trunc or not?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96037



More information about the llvm-commits mailing list