<div dir="ltr">Thanks for working on this!<br></div><br><div class="gmail_quote">чт, 23 апр. 2015 г. в 16:54, Daniel Sanders <<a href="mailto:Daniel.Sanders@imgtec.com">Daniel.Sanders@imgtec.com</a>>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">> -----Original Message-----<br>
> From: Timur Iskhodzhanov [mailto:<a href="mailto:timurrrr@google.com" target="_blank">timurrrr@google.com</a>]<br>
> Sent: 23 April 2015 14:38<br>
> To: Daniel Sanders; <a href="mailto:kcc@google.com" target="_blank">kcc@google.com</a>; Sagar Thakur; <a href="mailto:timurrrr@google.com" target="_blank">timurrrr@google.com</a><br>
> Cc: <a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
> Subject: Re: [PATCH] [asan] debug_mapping.cc should also pass when the<br>
> leading digit is hexadecimal.<br>
><br>
> Nice catch, I've overlooked that code!<br>
><br>
> `SHADOW_SCALE` is currently a constant equal to `3` and it can't be more<br>
> than 8 with the current ASan design, I think?<br>
<br>
Now that you mention it, I see there's a 'CHECK(SHADOW_SCALE >= 3 && SHADOW_SCALE <= 7);' a few lines down.<br>
<br>
> Probably, the same applies to `SHADOW_GRANULARITY` - it is currently equal<br>
> to 8, can't be more than 256 and I'd argue we should always print them out as<br>
> decimals for readability.<br>
><br>
> That's being said, I think the `Printf` line you're referring to should be<br>
> changed to just use `%d`.<br>
> [I suspect this is a side effect of copy-pasting code]<br>
<br>
That makes sense to me. I'll upload a patch for those two shortly.<br>
<br>
> <a href="http://reviews.llvm.org/D9199" target="_blank">http://reviews.llvm.org/D9199</a><br>
><br>
> EMAIL PREFERENCES<br>
>   <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
><br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</blockquote></div>