fix some bad logic that was removing all successor phi nodes, not just
authorJohn Regehr <regehr@cs.utah.edu>
Sun, 14 Aug 2022 01:14:10 +0000 (19:14 -0600)
committerJohn Regehr <regehr@cs.utah.edu>
Sun, 14 Aug 2022 01:15:26 +0000 (19:15 -0600)
commitdf308cab28bdf53bb159d2927fc25ffab5a79fe7
tree0578c326c1b4356ddbf8fe44e68d4102a1cf940b
parent83aa91bda0fae725e86982b408770f3f90bcca83
fix some bad logic that was removing all successor phi nodes, not just
out of chunk ones. the non-default second argument to
removePredecessor() is necessary to avoid creating invalid IR on
examples like the one in the provided test case

Differential Revision: https://reviews.llvm.org/D131843
llvm/test/tools/llvm-reduce/remove-bbs-nocrash.ll [new file with mode: 0644]
llvm/tools/llvm-reduce/deltas/ReduceBasicBlocks.cpp