[PATCH] D42728: Add more warnings for implict conversions (e.g. double truncation to float).

Alexey Bataev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 14 10:53:44 PDT 2018


ABataev added inline comments.


================
Comment at: lib/Sema/SemaChecking.cpp:9527
+static void CheckImplicitConversion(Sema &S, Expr *E, QualType T,
+                                    SourceLocation CC, bool *ICContext) {
   if (E->isTypeDependent() || E->isValueDependent()) return;
----------------
DO you still need `ICContext` parameter? I don't see where it is used.


https://reviews.llvm.org/D42728





More information about the llvm-commits mailing list