[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