[PATCH] D87033: [APInt] New member function setBitTo

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 2 09:56:27 PDT 2020


RKSimon added inline comments.


================
Comment at: llvm/include/llvm/ADT/APInt.h:1454
+  /// Set a given bit to a given value.
+  void setBitTo(unsigned BitPosition, bool BitValue) {
+    assert(BitPosition < BitWidth && "BitPosition out of range");
----------------
lebedev.ri wrote:
> I think `setBitVal()` might be better.
+1


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87033



More information about the llvm-commits mailing list