[clang] [C2y] Handle FP-suffixes on prefixed octals (#141230) (PR #141695)
Aaron Ballman via cfe-commits
cfe-commits at lists.llvm.org
Wed May 28 05:18:50 PDT 2025
================
@@ -1432,14 +1432,26 @@ void NumericLiteralParser::ParseNumberStartingWithZero(SourceLocation TokLoc) {
Diags.Report(TokLoc, DiagId);
++s;
DigitsBegin = s;
- SawOctalPrefix = true;
+ radix = 8;
----------------
AaronBallman wrote:
There's code below which also unconditionally sets `radix` to `8`; we should remove that and update the comments. We also seem to skip the octal digits twice (I think it's a noop to do it a second time, but it's still a code smell).
https://github.com/llvm/llvm-project/pull/141695
More information about the cfe-commits
mailing list