================
@@ -138,20 +152,28 @@ IntegerLiteralSeparatorFixer::process(const Environment
&Env,
Text = Text.substr(Start, Length);
}
auto DigitsPerGroup = Decimal;
- auto MinDigits = DecimalMinDigits;
+ auto MinDigitsInsert = DecimalMinDigitsInsert;
+ auto MaxDigitsRemove = DecimalMaxDigitsRemove;
if (IsBase2) {
DigitsPerGroup = Binary;
- MinDigits = BinaryMinDigits;
+ MinDigitsInsert = BinaryMinDigitsInsert;
+ MaxDigitsRemove = BinaryMaxDigitsRemove;
} else if (IsBase16) {
DigitsPerGroup = Hex;
- MinDigits = HexMinDigits;
+ MinDigitsInsert = HexMinDigitsInsert;
+ MaxDigitsRemove = HexMaxDigitsRemove;
}
const auto SeparatorCount = Text.count(Separator);
const int DigitCount = Length - SeparatorCount;
----------------
owenca wrote:
```suggestion
const int DigitCount = Length - SeparatorCount;
if (DigitCount > MaxDigitsRemove && DigitCount < MinDigitsInsert)
continue;
```
https://github.com/llvm/llvm-project/pull/164286
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
