[PATCH] D140059: [APSInt] Fix bug in APSInt mentioned in https://github.com/llvm/llvm-project/issues/59515

[Sergei Barannikov via Phabricator via llvm-commits]

barannikov88 added inline comments.


================
Comment at: llvm/include/llvm/ADT/APSInt.h:94
+    // not give them a vague value at the first place.
+    return (isSigned() && getMinSignedBits() <= 64) ||
+           (!isSigned() && getActiveBits() <= 63);
----------------
The comment near `getMinSignBits` says that it is deprecated (and "use `getSignificantBits()` instead").
There are also `isSignedIntN` and `isIntN`, which can be useful:
```
    return isSigned() ? isSignedIntN(64) : isIntN(63);
```



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140059