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

Sam Parker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 13 05:24:47 PST 2017


samparker created this revision.
samparker added a reviewer: niravd.
Herald added a subscriber: javed.absar.

https://reviews.llvm.org/rL319773 was reverted due to a recursive issue causing timeouts. This happened because I failed to check whether the discovered loads could be narrowed further. In the cases of narrow loads that could not be further narrowed, an AND was being introduced and not combined away - only to be combined into another AND and so on... forever.


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.126737.patch
Type: text/x-patch
Size: 39904 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171213/c4314855/attachment.bin>


More information about the llvm-commits mailing list