[PATCH] D86963: [DAGCombine] Skip re-visiting EntryToken to avoid compile time explosion

Ulrich Weigand via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 2 04:51:39 PDT 2020


uweigand added a comment.

In D86963#2250247 <https://reviews.llvm.org/D86963#2250247>, @fhahn wrote:

> Could you share the IR before DAGcombine that triggers the problem? DAGCombine already has limits in a few places that should prevent store merging from causing an explosion in compile-time. Would be good to know where this case slips through.

Sure, here's the IR of the function (which remains basically unchanged until ISel), and also the SelectionDAG of the basic block just before the DAGCombine that takes a very long time:
F12838384: matrix-test.ll <https://reviews.llvm.org/F12838384>
F12838386: matrix-test.isel <https://reviews.llvm.org/F12838386>


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86963



More information about the llvm-commits mailing list