[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