[PATCH] D23623: [asan] Add support of lifetime poisoning into ComputeASanStackFrameLayout

Evgeniy Stepanov via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 17 17:52:01 PDT 2016


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

LGTM


================
Comment at: lib/Transforms/Utils/ASanStackFrameLayout.cpp:84
@@ -83,2 +83,3 @@
     assert(Size > 0);
+    assert(Vars[i].LifetimeSize <= Size);
     StackDescription << " " << Offset << " " << Size << " " << strlen(Name)
----------------
You need a stronger assert: lifetime aligned up is not larger than size. Right?


https://reviews.llvm.org/D23623





More information about the llvm-commits mailing list