[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