[PATCH] D31567: MemorySSA: Add support for caching clobbering access in stores

[Davide Italiano via Phabricator via llvm-commits]

davide accepted this revision.
davide added a comment.
This revision is now accepted and ready to land.

LG



================
Comment at: lib/Transforms/Utils/MemorySSA.cpp:2202
     Cache.insert(StartingAccess, LiveOnEntry, Q.StartingLoc, Q.IsCall);
-    if (MemoryUse *MU = dyn_cast<MemoryUse>(StartingAccess))
-      MU->setDefiningAccess(LiveOnEntry, true);
+    if (MemoryUseOrDef *MUD = dyn_cast<MemoryUseOrDef>(StartingAccess))
+      MUD->setDefiningAccess(LiveOnEntry, true);
----------------
`auto`. I know it was already there, but while you're at it :)


================
Comment at: lib/Transforms/Utils/MemorySSA.cpp:2220
   DEBUG(dbgs() << *Result << "\n");
-  if (MemoryUse *MU = dyn_cast<MemoryUse>(StartingAccess))
-    MU->setDefiningAccess(Result, true);
+  if (MemoryUseOrDef *MUD = dyn_cast<MemoryUseOrDef>(StartingAccess))
+    MUD->setDefiningAccess(Result, true);
----------------
`auto`


https://reviews.llvm.org/D31567