[PATCH] D52827: [LICM] Make LICM able to hoist phis

[Max Kazantsev via Phabricator via llvm-commits]

mkazantsev requested changes to this revision.
mkazantsev added a comment.
This revision now requires changes to proceed.

One of my fuzz tests has failed with the following assertion:

  PHINode should have one entry for each predecessor of its parent basic block!
  %.us-phi = phi i32 [ %res.i.peel, %cHeapLvb.exit181 ], [ %83, %bci_229.licm.split.us.licm ], [ %83, %bci_229.licm.split.us.licm ]

It will take me time to reduce the test to something reasonable, but maybe this info will help you find the bug in the code. So far I can say that the problematic situation happens when `FalseDest == CommonSucc`. Hope if helps you to diagnose the bug and construct a test. If not, I'll be able to provide the reduced test case early next week.


Repository:
  rL LLVM

https://reviews.llvm.org/D52827