This is an archive of the discontinued LLVM Phabricator instance.

Fix compiler_builtins.m test to not rely on including system stdlib.h and malloc.h
ClosedPublic

Authored by hans on Jan 21 2015, 4:41 PM.

Details

Summary

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.

Diff Detail

Repository
rL LLVM

Event Timeline

hans updated this revision to Diff 18568.Jan 21 2015, 4:41 PM
hans retitled this revision from to Fix compiler_builtins.m test to not rely on including system stdlib.h and malloc.h.
hans updated this object.
hans edited the test plan for this revision. (Show Details)
hans added a reviewer: rsmith.
hans added a subscriber: Unknown Object (MLST).
rsmith accepted this revision.Jan 21 2015, 4:43 PM
rsmith edited edge metadata.

LGTM

This revision is now accepted and ready to land.Jan 21 2015, 4:43 PM
This revision was automatically updated to reflect the committed changes.