================ @@ -761,6 +803,17 @@ ExprResult Sema::ImpCastExprToType(Expr *E, QualType Ty, } } + // FIXME: Doesn't include C89, so this warning isn't emitted when passing + // `std=c89`. + auto isC = getLangOpts().C99 || getLangOpts().C11 || getLangOpts().C17 || ---------------- PhilippRados wrote:
> Why doesn't this make sense in ObjC? I thought that this was some kind of common pattern in ObjC which is why the existing `DiagnoseAssignmentAsCondition()` also special cases this. I think I might have also seen some tests that check this, but I could be wrong. https://github.com/llvm/llvm-project/pull/115234 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits