[PATCH] D85050: [GlobalISel] Combine (logic_op (op x...), (op y...)) -> (op (logic_op x, y))

Jessica Paquette via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 11 09:54:40 PDT 2020


paquette updated this revision to Diff 284800.
paquette added a comment.

- Add back tests
- Use `= default` for default constructors
- Add some asserts to `applyBuildInstructionSteps`


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

https://reviews.llvm.org/D85050

Files:
  llvm/include/llvm/CodeGen/GlobalISel/CombinerHelper.h
  llvm/include/llvm/Target/GlobalISel/Combine.td
  llvm/lib/CodeGen/GlobalISel/CombinerHelper.cpp
  llvm/lib/Target/AArch64/AArch64Combine.td
  llvm/test/CodeGen/AArch64/GlobalISel/postlegalizercombiner-hoist-same-hands.mir
  llvm/test/CodeGen/AArch64/GlobalISel/prelegalizercombiner-hoist-same-hands.mir

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D85050.284800.patch
Type: text/x-patch
Size: 34358 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200811/7c9667aa/attachment.bin>


More information about the llvm-commits mailing list