[PATCH] D91445: [APInt] Add the truncOrSelf resizing operator to APInt

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Nov 14 09:06:12 PST 2020


RKSimon accepted this revision.
RKSimon added a comment.
This revision is now accepted and ready to land.

LGTM with a couple of minors - cheers



================
Comment at: llvm/include/llvm/ADT/APInt.h:1398
+  /// truncated or left alone to make it that width.
+  APInt truncOrSelf(unsigned width) const;
+
----------------
Maybe move this down to be next to zextOrSelf/sextOrSelf?


================
Comment at: llvm/lib/Support/APInt.cpp:952
+  return *this;
+}
+
----------------
Maybe move this down to be next to zextOrSelf/sextOrSelf?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D91445/new/

https://reviews.llvm.org/D91445



More information about the llvm-commits mailing list