[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