[PATCH] D30221: Move updating functions to MemorySSAUpdater.Add updater to passes that now need it.Move around code in MemorySSA to expose needed functions.
Piotr Padlewski via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Feb 22 09:25:14 PST 2017
Prazek added inline comments.
================
Comment at: include/llvm/Transforms/Utils/MemorySSAUpdater.h:90-94
+ // The below are utility functions. Other than creation of accesses to pass
+ // to insertDef, and removeAccess to remove accesses, you should generally
+ // not attempt to update memoryssa yourself. It is very non-trivial to get
+ // the edge cases right, and the above calls already operate in near-optimal
+ // time bounds.
----------------
The plan is to remove these functions in the future right?
https://reviews.llvm.org/D30221
More information about the llvm-commits
mailing list