This is an archive of the discontinued LLVM Phabricator instance.

[openmp] [test] [runtime] Do not include alloca.h on NetBSD
ClosedPublic

Authored by mgorny on Dec 9 2018, 4:11 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

mgorny created this revision.Dec 9 2018, 4:11 AM

I propose to include (c)stdlib(.h) for everybody. And include alloca.h for everybody except NetBSD, FreeBSD, DragonFly and OpenBSD.

mgorny updated this revision to Diff 177429.Dec 9 2018, 8:07 AM

Updated as requested by @krytarowski.

krytarowski accepted this revision.Dec 9 2018, 8:21 AM
This revision is now accepted and ready to land.Dec 9 2018, 8:21 AM
This revision was automatically updated to reflect the committed changes.

Just to add to the conversation neither does FreeBSD.