[PATCH] D73032: [DependenceAnalysis] Memory dependence analysis internal caching mechanism is broken in presence of TBAA (PR42733).

John Brawn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 13 09:01:28 PST 2020


john.brawn added a comment.

It looks like the change that introduced isInvariantLoad was reverted in commit 446acafb82b5c116b6c94c11d4ac4db7641fa58d <https://reviews.llvm.org/rG446acafb82b5c116b6c94c11d4ac4db7641fa58d>, so it's a bit hard to review this with current trunk. Is there any problem with memory dependence analysis as it is, or is there a problem only with isInvariantLoad?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73032





More information about the llvm-commits mailing list