[PATCH] D58560: [AST] Set 'MayAlias' instead of 'MustAlias' in AliasSets for PHI nodes (bugzilla bug#36801)

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 27 16:41:30 PST 2019


efriedma added a comment.

I don't really like this even as a temporary fix.  Sure, if the PHI node is inside the loop, we're not optimizing it anyway, but I would expect it isn't uncommon to have a PHI node outside the loop, which this would block optimizing.  And I'm worried the SSAUpdater usage of ValueIsRAUWd could cause bugs elsewhere.


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

https://reviews.llvm.org/D58560





More information about the llvm-commits mailing list