[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