================

----------------
sdesmalen-arm wrote:

Would it have been possible to change ARMCPUTestParams to give it two 
constructors, one taking a uint64_t mask (for the ARM features), and the other 
constructor taking a BitVector (for the AArch64 features), and then to have the 
classes work on a BitVector under the hood?

For example, it's possible to convert a bitmask -> BitVector with a simple loop:
```uint64_t Mask = ... // the mask you want to convert to a BitVector
BitVector V(64);
for (unsigned I=0; I<64; ++I) {
  if ((Mask >> I) & 1)
    V.set(I);
}
```

https://github.com/llvm/llvm-project/pull/65423
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to