[PATCH] D144423: [APInt] Deprecate several functions

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: llvm/include/llvm/ADT/APInt.h:1559
 
+  LLVM_DEPRECATED("use countl_zero instead", "countl_zero")
   unsigned countLeadingZeros() const { return countl_zero(); }
----------------
I think this is a way worse name


================
Comment at: llvm/include/llvm/ADT/APInt.h:1578
 
+  LLVM_DEPRECATED("use countl_one instead", "countl_one")
   unsigned countLeadingOnes() const { return countl_one(); }
----------------
Ditto


================
Comment at: llvm/include/llvm/ADT/APInt.h:1602
 
+  LLVM_DEPRECATED("use countr_zero instead", "countr_zero")
   unsigned countTrailingZeros() const { return countr_zero(); }
----------------
Ditto


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144423