[PATCH] D86001: [MSAN] Avoid dangling ActualFnStart when replacing instruction

Evgenii Stepanov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 14 16:39:50 PDT 2020


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

LGTM with a nit



================
Comment at: llvm/lib/Transforms/Instrumentation/MemorySanitizer.cpp:1103
+      IRBuilder<> IRB(FnPrologueEnd);
+      FnPrologueEnd = insertKmsanPrologue(IRB);
     }
----------------
I don't think you need to update FnPrologueEnd anymore.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86001



More information about the llvm-commits mailing list