[PATCH] [asan] debug_mapping.cc should also pass when the leading digit is hexadecimal.
Timur Iskhodzhanov
timurrrr at google.com
Thu Apr 23 06:37:49 PDT 2015
Nice catch, I've overlooked that code!
`SHADOW_SCALE` is currently a constant equal to `3` and it can't be more than 8 with the current ASan design, I think?
Probably, the same applies to `SHADOW_GRANULARITY` - it is currently equal to 8, can't be more than 256 and I'd argue we should always print them out as decimals for readability.
That's being said, I think the `Printf` line you're referring to should be changed to just use `%d`.
[I suspect this is a side effect of copy-pasting code]
http://reviews.llvm.org/D9199
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list