riccibruno added inline comments.
================ Comment at: include/clang/Basic/Sanitizers.h:43 + // Low bits of mask value. + uint64_t maskLo; + // High bits of mask value. ---------------- Why not use a fixed size array of `uint64_t`s, and then compute the index without any branch with `ArrayIndex = BitPosition / 64` (with an assertion that `ArrayIndex < MaxArrayIndex` ? This has the advantage that in the future all that is needed to do is bump up the size of the array. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57914/new/ https://reviews.llvm.org/D57914 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
