<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 23 April 2015 at 15:32, Daniel Sanders <span dir="ltr"><<a href="mailto:daniel.sanders@imgtec.com" target="_blank">daniel.sanders@imgtec.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi kcc, timurrrr,<br>
<br>
During the review of <a href="http://reviews.llvm.org/D9199" target="_blank">http://reviews.llvm.org/D9199</a> where I had originally<br>
changed the debug_mapping.cc test to accept hexadecimal values, we realized<br>
that SHADOW_SCALE and SHADOW_GRANULARITY ought to be printed as decimal values.<br>
This patch makes that change.<br>
<br>
This patch also adds a '0x' prefix to the SHADOW_OFFSET to make it clear that<br>
it is hexadecimal while the other two are decimal.<br>
<br>
<a href="http://reviews.llvm.org/D9224" target="_blank">http://reviews.llvm.org/D9224</a><br>
<br>
Files:<br>
  lib/asan/asan_rtl.cc<br>
  test/asan/TestCases/debug_mapping.cc<br>
<br>
Index: lib/asan/asan_rtl.cc<br>
===================================================================<br>
--- lib/asan/asan_rtl.cc<br>
+++ lib/asan/asan_rtl.cc<br>
@@ -347,9 +347,9 @@<br>
   Printf("malloc_context_size=%zu\n",<br>
          (uptr)common_flags()->malloc_context_size);<br>
<br>
-  Printf("SHADOW_SCALE: %zx\n", (uptr)SHADOW_SCALE);<br>
-  Printf("SHADOW_GRANULARITY: %zx\n", (uptr)SHADOW_GRANULARITY);<br>
-  Printf("SHADOW_OFFSET: %zx\n", (uptr)SHADOW_OFFSET);<br>
+  Printf("SHADOW_SCALE: %d\n", (uptr)SHADOW_SCALE);<br></blockquote><div>If it's cast to uptr, then surely it should be "%zd" or even "%zu", not "%d" (the z means "size_t" length of the value)<br><br>--<br></div><div>Mats<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+  Printf("SHADOW_GRANULARITY: %d\n", (uptr)SHADOW_GRANULARITY);<br>
+  Printf("SHADOW_OFFSET: 0x%zx\n", (uptr)SHADOW_OFFSET);<br>
   CHECK(SHADOW_SCALE >= 3 && SHADOW_SCALE <= 7);<br>
   if (kMidMemBeg)<br>
     CHECK(kMidShadowBeg > kLowShadowEnd && <br></blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Index: test/asan/TestCases/debug_mapping.cc<br>
===================================================================<br>
--- test/asan/TestCases/debug_mapping.cc<br>
+++ test/asan/TestCases/debug_mapping.cc<br>
@@ -8,14 +8,14 @@<br>
<br>
 // printed because of verbosity=1<br>
 // CHECK: SHADOW_SCALE: [[SCALE:[0-9]+]]<br>
-// CHECK: SHADOW_OFFSET: [[OFFSET:[0-9a-f]+]]<br>
+// CHECK: SHADOW_OFFSET: [[OFFSET:0x[0-9a-f]+]]<br>
<br>
 int main() {<br>
   size_t scale, offset;<br>
   __asan_get_shadow_mapping(&scale, &offset);<br>
<br>
-  fprintf(stderr, "scale: %lx\n", scale);<br>
-  fprintf(stderr, "offset: %lx\n", offset);<br>
+  fprintf(stderr, "scale: %d\n", scale);<br></blockquote><div>Likewise size_t should be "%zd" [and for the hex on the next line "%zx", not "%lx"]<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+  fprintf(stderr, "offset: 0x%lx\n", offset);<br>
<br>
   // CHECK: scale: [[SCALE]]<br>
   // CHECK: offset: [[OFFSET]]<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>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">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>
<br></blockquote></div><br></div></div>