[PATCH] D33587: [DAGCombine] Do several rounds of combine.

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jan 26 20:47:22 PST 2019


craig.topper added a comment.

Currently we largely visit nodes before their operands. So when we simplify the operands several layers down we don’t revisit the later nodes to match patterns. But if we visited the operands first, then the first visit of the later nodes would occur after. So we wouldn’t need to revisit them.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D33587





More information about the llvm-commits mailing list