[PATCH] D33716: Provide path to Solaris.h include (Bug 28370)
Fedor Sergeev via llvm-commits
llvm-commits at lists.llvm.org
Sat Jun 10 01:27:54 PDT 2017
On Sat, Jun 10, 2017 at 08:45:20AM +0200, Joerg Sonnenberger wrote:
> On Sat, Jun 10, 2017 at 01:07:54AM +0300, Fedor Sergeev wrote:
> > On Fri, Jun 09, 2017 at 11:58:10PM +0200, Joerg Sonnenberger via llvm-commits wrote:
> > > A sys/regset.h wrapper using #include_next might work just as well, if
> > > this is the only reason.
> >
> > This wont work for system includes unless you have a full control over system include path
> > (with something like -isystem).
> > And that just switches the problem from -include to -isystem.
>
> As long as the Solaris headers use <sys/regset.h> as well and not depend
> on the relative locations with "", it works well. That's what we do in
> NetBSD for certain compat overrides for build tools.
Okey, you dont need -isystem, but you still need to provide platform-specific -I.
I will try crafting something like that and see how it copes with LLVM build system.
regards,
Fedor.
More information about the llvm-commits
mailing list