[PATCH] D115167: [DSE] Use precise loc for memset_chk during overwrite checks

[Alina Sbirlea via Phabricator via llvm-commits]

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

Considering the previous reviewer comments, this lgtm.



================
Comment at: llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp:886
+      if (TLI.getLibFunc(*CB, F) && TLI.has(F) && F == LibFunc_memset_chk) {
+        // Use the precise location size specified by the 3rd argument during
+        // for determining KillingI overwrites DeadLoc if it is a memset_chk
----------------
nit: remove 'during'


================
Comment at: llvm/test/Transforms/DeadStoreElimination/libcalls.ll:446
 
 define void @dse_memset_chk_cannot_eliminates_store(ptr %out, i64 %n) {
 ; CHECK-LABEL: @dse_memset_chk_cannot_eliminates_store(
----------------
nit: rename function without 'cannot'


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115167