nickdesaulniers added inline comments.
================
Comment at: clang/lib/Sema/SemaExpr.cpp:14539
LangOptions LO = S.getLangOpts();
- if (LO.CPlusPlus || !(LO.C99 && IsAfterAmp))
+ if (LO.CPlusPlus || !(LO.C99 && (IsAfterAmp || S.isUnevaluatedContext())))
S.Diag(OpLoc, diag::ext_typecheck_indirection_through_void_pointer)
----------------
nickdesaulniers wrote:
> Sorry, I should have asked more about this subexpresssion.
>
> Our CI is still red for older branches of the Linux kernel ("stable"
> branches) because the kernel only recently started using `-std=gnu11` from
> `-std=gnu89`.
>
> Why is this considered an error for `-std=gnu89`?
https://reviews.llvm.org/D135090
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D134702/new/
https://reviews.llvm.org/D134702
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
