[PATCH] D49945: [Fixed Point Arithmetic] Fix for FixedPointValueToString

Bevin Hansson via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 6 04:37:23 PDT 2018


ebevhan added inline comments.


================
Comment at: lib/AST/Expr.cpp:788
   FixedPointValueToString(
-      S, llvm::APSInt::getUnsigned(getValue().getZExtValue()), Scale, Radix);
+      S, llvm::APSInt::getUnsigned(getValue().getZExtValue()), Scale);
   return S.str();
----------------
Unrelated to this patch specifically, but using `getZExtValue` here is a bit limiting.


Repository:
  rC Clang

https://reviews.llvm.org/D49945





More information about the cfe-commits mailing list