[PATCH] D114872: [MemoryLocation] Move DSE's logic to new MemLoc::getForDest helper (NFC).

[Philip Reames via Phabricator via llvm-commits]

reames added inline comments.


================
Comment at: llvm/lib/Analysis/MemoryLocation.cpp:143
+    case LibFunc_strncat:
+      return getForArgument(CB, 0, &TLI);
+    default:
----------------
I might be missing something, but it looks like these are only handled generically inside MemoryLocation::getForArgument at the moment.  Given the original code specialized for the constant case, and only returned getAfter (not also before), I think this move is not NFC.  


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114872