[PATCH] D45910: [BasicAA] Don't assume a PHI node has inputs

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 20 22:33:18 PDT 2018


craig.topper added a comment.

Sorry I meant it became unreachable during jump threading. And jump threading itself created the degenerate phi. The call into alias analysis also came from jump threading from SimplifyPartiallyRedundantLoad.

But we do have our own internal changes in jump threading so its still possible that this is a problem there.

I found other code that handles phis with no incoming values in the phi handling in computeKnownBitsFromOperator. That code is exercised by 3 lit tests, 2 of them due to jump threading.


https://reviews.llvm.org/D45910





More information about the llvm-commits mailing list