[PATCH] D8688: Update MergedLoadStoreMotion to use MemorySSA
Geoff Berry via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 5 08:48:22 PDT 2016
gberry added a comment.
Sorry if this isn't the right place, but I have a general question about what it means to preserve MemorySSA, specifically regarding the defining accesses of MemoryUse nodes. Is the idea here that we make a best effort to keep the MemoryUse defining access links optimized (i.e. never pointing to a no-alias def)? Because of the limits of basic-aa, it isn't possible to guarantee this property after any code transformation, even in the limited case here, since the alias results for completely unrelated load/stores may have been affected, right?
More information about the llvm-commits