[PATCH] D57875: [LegalizeTypes] Expand FNEG to bitwise op for IEEE FP types

Ana Pazos via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 8 14:56:24 PST 2019


apazos added a comment.

Without the patch, the fneg use in the given test case becomes a sub lib call in the targets I checked  when using soft abi (riscv, arm)

DAGCombiner::visitFNEG is transforming fneg(bitconvert(x)) to bitwise operation. So for other patterns, we end up with a lib call which is wasteful.

I also need someone to confirm whether for x86 and ppc FP types it is valid to use bitwise operation. The patch now excludes these formats.


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

https://reviews.llvm.org/D57875





More information about the llvm-commits mailing list