[PATCH] D130155: [DeadStoreElimination] Handle null accessing

[Chuanqi Xu via Phabricator via llvm-commits]

ChuanqiXu added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp:1927
           if (auto *SI = dyn_cast<StoreInst>(DefInst)) {
-            // MemSetInst must have a write location.
-            MemoryLocation UpperLoc = *getLocForWrite(UpperInst);
----------------
Since there is a comment. I am not sure if this is a good fix.


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

https://reviews.llvm.org/D130155