[PATCH] D157767: [Driver] move Haiku header search path management to the driver

Brad Smith via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 11 18:34:38 PDT 2023


brad created this revision.
brad added a reviewer: nielx.
brad added a project: clang.
Herald added a project: All.
brad requested review of this revision.
Herald added subscribers: cfe-commits, MaskRay.

Here is a first cut at moving the Haiku header search path management to the driver as has been done for many other OS's to date.

InitHeaderSearch::AddDefaultCIncludePaths() currently does not skip adding /usr/local/include as well as /usr/include on Haiku. I installed
Haiku in a VM and see that neither path exists on an initial install. The other header paths and contents do exist out of the box. I have
not included those paths when moving the header path management to the driver so far.

I have not  built this on Haiku. I am looking for an actual Haiku user to test this.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D157767

Files:
  clang/lib/Driver/ToolChains/Haiku.cpp
  clang/lib/Driver/ToolChains/Haiku.h
  clang/lib/Lex/InitHeaderSearch.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D157767.549559.patch
Type: text/x-patch
Size: 8385 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20230812/36be5751/attachment.bin>


More information about the cfe-commits mailing list