[PATCH] D52835: [Diagnostics] Check integer to floating point number implicit conversions
Eli Friedman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Oct 11 12:49:34 PDT 2018
efriedma added inline comments.
================
Comment at: lib/Sema/SemaChecking.cpp:10881-10882
+ S.DiagRuntimeBehavior(
+ E->getExprLoc(), E,
+ S.PDiag(diag::warn_impcast_literal_float_to_integer)
+ << E->getType() << T << PrettySourceValue << PrettyTargetValue
----------------
It looks like this is reusing an existing diagnostic; what warning flag is this controlled by? It it possible we need a separate flag for this warning?
https://reviews.llvm.org/D52835
More information about the cfe-commits
mailing list