[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