[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 15:01:13 PST 2019


efriedma added a comment.

> Then LoopRotate calls the above magic using the SSAUpdater, "informing" the cached AliasSetTracker first that %p.0 is replaced with null and then that %p.0  is replaced with %arrayidx

Calling ValueIsRAUWd (allUsesReplacedWith) when the operation isn't equivalent to an RAUW seems dubious in general; maybe we should add a new callback?  Haven't really looked closely at what other implementations of allUsesReplacedWith are doing.


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

https://reviews.llvm.org/D58560





More information about the llvm-commits mailing list