[clang] [clang] Emit bad shift warnings (PR #70307)

Aaron Ballman via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 19 05:50:38 PDT 2024


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


More information about the cfe-commits mailing list