[PATCH] D59315: [AliasAnalysis] Second prototype to cache BasicAA / anyAA state.

George Burgess IV via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 20 19:41:59 PDT 2019


george.burgess.iv added a comment.

> The reason is that MemorySSA has to be moveable

After looking through logs, I take this back -- the verification is that MSSA is *never* relocated, and the verification was added in https://reviews.llvm.org/D23171 (maybe it was the Updater that we made move-able? I don't remember). In any case, it'd probably be a good idea to explicitly delete MSSA's move ctor to help ensure this move doesn't happen. I'll commit that shortly if it gives me no build errors.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D59315





More information about the llvm-commits mailing list