[PATCH] D86393: [GISel] Add combines for unary FP instrs with constant operand

Michael Kitzan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 4 17:12:22 PDT 2020


mkitzan updated this revision to Diff 290052.
mkitzan added a comment.

- Collapsed the many combine rules into a single combine rule with a list of matchable opcodes
- Made `default` switch case in `constantFoldFpUnary` be `llvm_unreachable`


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

https://reviews.llvm.org/D86393

Files:
  llvm/include/llvm/CodeGen/GlobalISel/CombinerHelper.h
  llvm/include/llvm/CodeGen/LowLevelType.h
  llvm/include/llvm/Target/GlobalISel/Combine.td
  llvm/lib/CodeGen/GlobalISel/CombinerHelper.cpp
  llvm/lib/CodeGen/LowLevelType.cpp
  llvm/test/CodeGen/AArch64/GlobalISel/combine-fabs.mir
  llvm/test/CodeGen/AArch64/GlobalISel/combine-flog2.mir
  llvm/test/CodeGen/AArch64/GlobalISel/combine-fneg.mir
  llvm/test/CodeGen/AArch64/GlobalISel/combine-fptrunc.mir
  llvm/test/CodeGen/AArch64/GlobalISel/combine-fsqrt.mir

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D86393.290052.patch
Type: text/x-patch
Size: 17666 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200905/bdf871aa/attachment.bin>


More information about the llvm-commits mailing list