AaronBallman wrote:

> > > > Yes and no. It's due to the redefinitions, for sure. But that 
> > > > diagnostic is suppressed in system headers: 
> > > > https://godbolt.org/z/Mb7Kh975f, so I think we need to understand why 
> > > > there's a `limits.h` being included as though it were not a system 
> > > > header, because that's a bug.
> > > 
> > > 
> > > As we are building `glibc`, in which it includes the `limits.h` of 
> > > itself, instead of the system one.
> > 
> > 
> > That's the bug then. It needs to include the header as a system header, not 
> > a user header.
> 
> Yes. It does. It uses `-I` option to include limits.h of itself. 

Do you get the same behavior when you use `-isystem` instead?



https://github.com/llvm/llvm-project/pull/120526
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to