[PATCH] D44564: [BasicAA] Relax restriction on PHI node handling.

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 25 12:43:34 PDT 2018


efriedma added a comment.

Hmm.  I don't see anything wrong with the code, but how often will this trigger?  instcombine will remove the PHI nodes in your testcase, so this is only a problem if you have a pass which creates PHI cycles like this.  I guess GVN does in some cases?  Can we avoid doing that?


Repository:
  rL LLVM

https://reviews.llvm.org/D44564





More information about the llvm-commits mailing list