[PATCH] D127115: [DAGCombine] Make sure combined nodes are added back to the worklist in topological order.

Amaury SECHET via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 7 17:46:32 PDT 2023


deadalnix added a comment.

I have this one, which repro and is simpler:

  define i64 @foo(i1 %0) {
    br label %2
  
  2:
    %3 = select i1 %0, i1 %0, i1 false
    %4 = freeze i1 %3
    br i1 %4, label %5, label %6
  
  5:
    br label %6
  
  6:
    %7 = phi i64 [ 0, %5 ], [ 1, %2 ]
    ret i64 %7
  }

I'll figure out a fix. That doesn't sound complicated.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127115



More information about the llvm-commits mailing list