[clang] [clang] Emit bad shift warnings (PR #70307)
Alan Zhao via cfe-commits
cfe-commits at lists.llvm.org
Fri Jul 12 11:09:23 PDT 2024
alanzhao1 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.
> > @budimirarandjelovicsyrmia, can you please add a workaround so that the NDK code can continue to compile?
>
> I will investigate this.
FYI I noticed that gcc can compile this with `-fpermissive`
https://github.com/llvm/llvm-project/pull/70307
More information about the cfe-commits
mailing list