[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