[PATCH] D35160: Only check shadow memory for operands that are sized.

Evgenii Stepanov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 10 15:50:16 PDT 2017


eugenis added inline comments.


================
Comment at: llvm/test/Instrumentation/MemorySanitizer/unsized_type.ll:18
+
+;attributes #1 = { argmemonly nounwind readonly }
+;attributes #2 = { nounwind }
----------------
eugenis wrote:
> remove the comment lines
In fact, the assertion failure is not really the point of this test. Please make it a positive one - check the instrumentation (or lack of it) of @foo. 


https://reviews.llvm.org/D35160





More information about the llvm-commits mailing list