[PATCH] D125557: [APInt] Remove all uses of zextOrSelf, sextOrSelf and truncOrSelf

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 17 08:45:47 PDT 2022


reames added a comment.

Coming into this late, but I'd have preferred to see this separated into at least two pieces.  One for each "non-obvious" adjustment, and one final one which just did the replace on the renaming sites.  This differs from feedback from other reviewers above, so don't feel bound by this in any way.  Just expressing the general preference.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125557



More information about the llvm-commits mailing list