[PATCH] D34827: Add end-to-end tests for overflows of byval arguments.

Matt Morehouse via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 13 14:25:00 PDT 2017


morehouse updated this revision to Diff 106533.
morehouse added a comment.
Herald added a subscriber: srhines.

Change UNSUPPORTED option to REQUIRES.  Test now works on x86_64 Linux and shouldn't cause the Android/ARM/Windows buildbots to fail.


https://reviews.llvm.org/D34827

Files:
  compiler-rt/test/asan/TestCases/pass-struct-byval-uar.cc


Index: compiler-rt/test/asan/TestCases/pass-struct-byval-uar.cc
===================================================================
--- compiler-rt/test/asan/TestCases/pass-struct-byval-uar.cc
+++ compiler-rt/test/asan/TestCases/pass-struct-byval-uar.cc
@@ -9,7 +9,7 @@
 // instead creates a copy in main() and gives foo() a pointer to the copy.  In
 // that case, ASAN has nothing to poison on return from foo() and will not
 // detect the UAR.
-// UNSUPPORTED: *
+// REQUIRES: x86_64-target-arch, linux, not-android
 
 #include <cstdio>
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D34827.106533.patch
Type: text/x-patch
Size: 541 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170713/3d48cbad/attachment.bin>


More information about the llvm-commits mailing list