[PATCH] D78145: [ValueLattice] Steal bits from Tag to track range extensions (NFC).

[Florian Hahn via Phabricator via llvm-commits]

fhahn marked an inline comment as done.
fhahn added inline comments.


================
Comment at: llvm/include/llvm/Analysis/ValueLattice.h:78
 
-  ValueLatticeElementTy Tag;
+  ValueLatticeElementTy Tag : 6;
+  /// Number of times a constant range has been extended with widening enabled.
----------------
nikic wrote:
> Why 6? I would expect either 3 here (because that's enough to represent the states) or 8 (an ISA friendly number).
The main intention was to leave enough space for plenty of further extension while keeping plenty of other bits for other stuff. But it's probably worth to extend it to 8. I'll do that as a follow up commit.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78145