[PATCH] Fix compiler_builtins.m test to not rely on including system stdlib.h and malloc.h
Hans Wennborg
hans at chromium.org
Wed Jan 21 16:41:32 PST 2015
Hi rsmith,
Importing _Builtin_intrinsics.sse and avx would transitively pull in those
headers, and the test would fail when building in an environment where
they were not available on the include path.
This fixes PR20995 for me, and I'd like to get this onto the 3.6 branch.
http://reviews.llvm.org/D7112
Files:
test/Modules/Inputs/System/usr/include/malloc.h
test/Modules/Inputs/System/usr/include/stdlib.h
test/Modules/compiler_builtins.m
Index: test/Modules/Inputs/System/usr/include/stdlib.h
===================================================================
--- /dev/null
+++ test/Modules/Inputs/System/usr/include/stdlib.h
@@ -0,0 +1 @@
+typedef __SIZE_TYPE__ size_t;
Index: test/Modules/compiler_builtins.m
===================================================================
--- test/Modules/compiler_builtins.m
+++ test/Modules/compiler_builtins.m
@@ -1,6 +1,6 @@
// RUN: rm -rf %t
-// RUN: %clang -fsyntax-only -fmodules -fmodules-cache-path=%t -D__need_wint_t %s -Xclang -verify
-// RUN: %clang -fsyntax-only -std=c99 -fmodules -fmodules-cache-path=%t -D__need_wint_t %s -Xclang -verify
+// RUN: %clang -fsyntax-only -fmodules -fmodules-cache-path=%t -D__need_wint_t %s -I%S/Inputs/System/usr/include -Xclang -verify
+// RUN: %clang -fsyntax-only -std=c99 -fmodules -fmodules-cache-path=%t -D__need_wint_t %s -I%S/Inputs/System/usr/include -Xclang -verify
// expected-no-diagnostics
#ifdef __SSE__
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D7112.18568.patch
Type: text/x-patch
Size: 973 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150122/4439a812/attachment.bin>
More information about the cfe-commits
mailing list