AaronBallman wrote:

> > > @alanzhao1 do you think it's reasonable for the workaround to only apply 
> > > to code in system headers, or does the NDK get included as regular 
> > > headers generally?
> > 
> > 
> > Having it only apply to system headers should be OK - in our case, chrome 
> > imports these headers as part of a `--sysroot`, which IIRC treats warnings 
> > the same way as `-isystem`
> > > Do you need us to revert the changes while we resolve this?
> > 
> > 
> > If you can revert while working on a solution, that would be helpful.
> 
> We (Chrome) no longer need a revert - we patched the NDK locally.

Thank you for letting us know!

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

Reply via email to