[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