[PATCH] D32155: [APInt] Use lshrInPlace to replace lshr where possible

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added inline comments.


================
Comment at: include/llvm/ADT/APInt.h:892
   /// Logical right-shift this APInt by ShiftAmt in place.
-  void lshrInPlace(unsigned ShiftAmt);
+  APInt &lshrInPlace(unsigned ShiftAmt);
 
----------------
Other than operators we don't tend to use this pattern to return *this - is it a good idea to introduce it?


https://reviews.llvm.org/D32155