[PATCH] D41177: [DAGCombine] Move AND nodes to multiple load leaves

Sam Parker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 15 08:15:24 PST 2017


samparker updated this revision to Diff 127137.
samparker added a comment.

https://reviews.llvm.org/rL320679 was reverted due to a miscompilation. I had made the false assumption that constant operands would have been narrowed. So now any OR and XOR nodes which use constant operands, which are wider than the mask, have their operand narrowed at the end of the process.


https://reviews.llvm.org/D41177

Files:
  lib/CodeGen/SelectionDAG/DAGCombiner.cpp
  test/CodeGen/ARM/and-load-combine.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D41177.127137.patch
Type: text/x-patch
Size: 43720 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171215/424bb433/attachment.bin>


More information about the llvm-commits mailing list