[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