[PATCH] D73342: Fix EarlyCSE to intersect aliasing metadata.
Nuno Lopes via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jan 24 08:14:00 PST 2020
nlopes added a comment.
I'm not familiar with the code of this pass, but is there a cheap way of identifying that the two operations are in the same basic block?
If so, you could take the intersection of the aliasing information rather than the union. Because if both ops are guaranteed to execute then the tightest aliasing still has to hold.
(being in the same BB doesn't imply that both instructions are executed, but there's code in ValueTracking perhaps that can check that)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D73342/new/
https://reviews.llvm.org/D73342
More information about the llvm-commits
mailing list