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

Leonard Chan via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 27 15:25:11 PDT 2018


leonardchan created this revision.
leonardchan added reviewers: phosek, mcgrathr, jakehehrlich, ebevhan, rjmccall.
leonardchan added a project: clang.
Herald added a subscriber: mgorny.

- Print negative numbers correctly
- Handle APInts of different sizes
- Add formal unit tests for FixedPointValueToString
- Add tests for checking correct printing when padding is set


Repository:
  rC Clang

https://reviews.llvm.org/D49945

Files:
  include/clang/AST/Type.h
  lib/AST/Expr.cpp
  lib/AST/ExprConstant.cpp
  lib/AST/Type.cpp
  test/Frontend/fixed_point_to_string.c
  unittests/Frontend/CMakeLists.txt
  unittests/Frontend/FixedPointString.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D49945.157787.patch
Type: text/x-patch
Size: 8276 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180727/73013fd5/attachment.bin>


More information about the cfe-commits mailing list