[PATCH] D130557: [Support] Add KnownBits::concatBits helper

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added inline comments.


================
Comment at: llvm/include/llvm/Support/KnownBits.h:309
+  static KnownBits concatBits(const KnownBits &Lo, const KnownBits &Hi) {
+    unsigned LoBits = Lo.getBitWidth();
+    unsigned HiBits = Hi.getBitWidth();
----------------
foad wrote:
> Maybe just `return { Hi.Zero.concat(Lo.Zero), Hi.One.concat(Lo.One) }`?
Hmm - I'd forgotten about APInt::concat - I guess we should try to make a similar KnownBits::concat ? We do tend to make them match where possible


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130557