[PATCH] D69439: [APInt] Introduce APIntOps::GetPositionOfMostSignificantDifferentBit()

Dávid Bolvanský via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Oct 26 09:44:12 PDT 2019


xbolva00 added inline comments.


================
Comment at: llvm/include/llvm/ADT/APInt.h:2227
+Optional<unsigned> GetPositionOfMostSignificantDifferentBit(const APInt &A,
+                                                            const APInt &B);
+
----------------
nikic wrote:
> The name seems a bit excessive -- I think `GetMostSignificantDifferingBit` would have the same information content.
or GetPosOfMostSignificantDifferentBit


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69439





More information about the llvm-commits mailing list