[PATCH] D52730: [analyzer] ConversionChecker: handle floating point
Artem Dergachev via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Oct 19 14:45:48 PDT 2018
NoQ added inline comments.
================
Comment at: lib/StaticAnalyzer/Checkers/ConversionChecker.cpp:163
+
+ switch (FloatingSize) {
+ case 64:
----------------
Continuing the float semantics discussion on the new revision - Did you consider `llvm::APFloat`? (http://llvm.org/doxygen/classllvm_1_1APFloat.html) This looks like something that you're trying to re-implement.
Repository:
rC Clang
https://reviews.llvm.org/D52730
More information about the cfe-commits
mailing list