leonardchan marked an inline comment as done.
leonardchan 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();
----------------
ebevhan wrote:
> Unrelated to this patch specifically, but using `getZExtValue` here is a bit 
> limiting.
I'll make another small patch to change this since should also do signed values 
now,


Repository:
  rC Clang

https://reviews.llvm.org/D49945



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to