Author: dirk Date: 2007-10-05 00:12:12 +0100 (Fri, 05 Oct 2007) New Revision: 6951
Log: fix format strings for the tools (except for helgrind) Modified: branches/FORMATCHECK/callgrind/bb.c branches/FORMATCHECK/callgrind/bbcc.c branches/FORMATCHECK/callgrind/callstack.c branches/FORMATCHECK/callgrind/debug.c branches/FORMATCHECK/callgrind/dump.c branches/FORMATCHECK/callgrind/fn.c branches/FORMATCHECK/callgrind/main.c branches/FORMATCHECK/callgrind/sim.c branches/FORMATCHECK/memcheck/mc_leakcheck.c branches/FORMATCHECK/memcheck/mc_main.c Modified: branches/FORMATCHECK/callgrind/bb.c =================================================================== --- branches/FORMATCHECK/callgrind/bb.c 2007-10-04 22:48:53 UTC (rev 6950) +++ branches/FORMATCHECK/callgrind/bb.c 2007-10-04 23:12:12 UTC (rev 6951) @@ -187,7 +187,7 @@ bb = bb->next; } - CLG_DEBUG(5, " lookup_bb (Obj %s, off %p): %p\n", + CLG_DEBUG(5, " lookup_bb (Obj %s, off %#lx): %p\n", obj->name, offset, bb); return bb; } @@ -207,7 +207,7 @@ if (obj->offset != offset) { Addr start = si ? VG_(seginfo_start)(si) : 0; - CLG_DEBUG(0, "Mapping changed for '%s': %p -> %p\n", + CLG_DEBUG(0, "Mapping changed for '%s': %#lx -> %#lx\n", obj->name, obj->start, start); /* Size should be the same, and offset diff == start diff */ @@ -242,7 +242,7 @@ UInt n_instrs, n_jmps; Bool cjmp_inverted = False; - CLG_DEBUG(5, "+ get_bb(BB %p)\n", addr); + CLG_DEBUG(5, "+ get_bb(BB %#lx)\n", addr); obj = obj_of_address(addr); bb = lookup_bb(obj, addr - obj->offset); @@ -269,13 +269,13 @@ CLG_ASSERT(bb->cjmp_count == n_jmps ); CLG_(stat).bb_retranslations++; - CLG_DEBUG(5, "- get_bb(BB %p): seen before.\n", addr); + CLG_DEBUG(5, "- get_bb(BB %#lx): seen before.\n", addr); return bb; } bb = new_bb(obj, addr - obj->offset, n_instrs, n_jmps, cjmp_inverted); - CLG_DEBUG(5, "- get_bb(BB %p)\n", addr); + CLG_DEBUG(5, "- get_bb(BB %#lx)\n", addr); return bb; } @@ -303,7 +303,7 @@ } if (bb == NULL) { - CLG_DEBUG(3, " delete_bb (Obj %s, off %p): NOT FOUND\n", + CLG_DEBUG(3, " delete_bb (Obj %s, off %lx): NOT FOUND\n", obj->name, offset); /* we didn't find it. @@ -324,7 +324,7 @@ bp->next = bb->next; } - CLG_DEBUG(3, " delete_bb (Obj %s, off %p): %p, BBCC head: %p\n", + CLG_DEBUG(3, " delete_bb (Obj %s, off %#lx): %p, BBCC head: %p\n", obj->name, offset, bb, bb->bbcc_list); if (bb->bbcc_list == 0) { Modified: branches/FORMATCHECK/callgrind/bbcc.c =================================================================== --- branches/FORMATCHECK/callgrind/bbcc.c 2007-10-04 22:48:53 UTC (rev 6950) +++ branches/FORMATCHECK/callgrind/bbcc.c 2007-10-04 23:12:12 UTC (rev 6951) @@ -85,7 +85,7 @@ jCC* jcc; CLG_ASSERT(bbcc->cxt != 0); - CLG_DEBUG(1, " zero_bbcc: BB %p, Cxt %d " + CLG_DEBUG(1, " zero_bbcc: BB %lx, Cxt %d " "(fn '%s', rec %d)\n", bb_addr(bbcc->bb), bbcc->cxt->base_number + bbcc->rec_index, @@ -177,7 +177,7 @@ bbcc = bbcc->next; } - CLG_DEBUG(2," lookup_bbcc(BB %p, Cxt %d, fn '%s'): %p (tid %d)\n", + CLG_DEBUG(2," lookup_bbcc(BB %lx, Cxt %d, fn '%s'): %p (tid %d)\n", bb_addr(bb), cxt->base_number, cxt->fn[0]->name, bbcc, bbcc ? bbcc->tid : 0); @@ -294,7 +294,7 @@ CLG_(stat).distinct_bbccs++; - CLG_DEBUG(3, " new_bbcc(BB %p): %p (now %d)\n", + CLG_DEBUG(3, " new_bbcc(BB %lx): %p (now %d)\n", bb_addr(bb), new, CLG_(stat).distinct_bbccs); return new; @@ -319,7 +319,7 @@ CLG_ASSERT(bbcc->cxt != 0); - CLG_DEBUG(3,"+ insert_bbcc_into_hash(BB %p, fn '%s')\n", + CLG_DEBUG(3,"+ insert_bbcc_into_hash(BB %lx, fn '%s')\n", bb_addr(bbcc->bb), bbcc->cxt->fn[0]->name); /* check fill degree of hash and resize if needed (>90%) */ @@ -368,7 +368,7 @@ { BBCC* new; - CLG_DEBUG(3,"+ clone_bbcc(BB %p, rec %d, fn %s)\n", + CLG_DEBUG(3,"+ clone_bbcc(BB %lx, rec %d, fn %s)\n", bb_addr(orig->bb), rec_index, cxt->fn[0]->name); new = new_bbcc(orig->bb); @@ -410,7 +410,7 @@ CLG_DEBUGIF(3) CLG_(print_bbcc)(-2, new, False); - CLG_DEBUG(2,"- clone_BBCC(%p, %d) for BB %p\n" + CLG_DEBUG(2,"- clone_BBCC(%p, %d) for BB %#lx\n" " orig %s\n" " new %s\n", orig, rec_index, bb_addr(orig->bb), @@ -433,7 +433,7 @@ { BBCC* bbcc; - CLG_DEBUG(3, "+ get_bbcc(BB %p)\n", bb_addr(bb)); + CLG_DEBUG(3, "+ get_bbcc(BB %#lx)\n", bb_addr(bb)); bbcc = bb->bbcc_list; @@ -453,7 +453,7 @@ CLG_(print_bbcc)(-2, bbcc, False); } - CLG_DEBUG(3, "- get_bbcc(BB %p): BBCC %p\n", + CLG_DEBUG(3, "- get_bbcc(BB %#lx): BBCC %p\n", bb_addr(bb), bbcc); return bbcc; @@ -558,7 +558,7 @@ Bool ret_without_call = False; Int popcount_on_return = 1; - CLG_DEBUG(3,"+ setup_bbcc(BB %p)\n", bb_addr(bb)); + CLG_DEBUG(3,"+ setup_bbcc(BB %#lx)\n", bb_addr(bb)); /* This is needed because thread switches can not reliable be tracked * with callback CLG_(run_thread) only: we have otherwise no way to get @@ -714,7 +714,7 @@ else ppIRJumpKind( jmpkind ); - VG_(printf)(" %08lx -> %08x, SP %08x\n", + VG_(printf)(" %08lx -> %08lx, SP %08lx\n", last_bb ? bb_jmpaddr(last_bb) : 0, bb_addr(bb), sp); } @@ -870,7 +870,7 @@ VG_(printf)("\n"); } - CLG_DEBUG(3,"- setup_bbcc (BB %p): Cost %p (Len %d), Instrs %d (Len %d)\n", + CLG_DEBUG(3,"- setup_bbcc (BB %#lx): Cost %p (Len %d), Instrs %d (Len %d)\n", bb_addr(bb), bbcc->cost, bb->cost_count, bb->instr_count, bb->instr_len); CLG_DEBUGIF(3) Modified: branches/FORMATCHECK/callgrind/callstack.c =================================================================== --- branches/FORMATCHECK/callgrind/callstack.c 2007-10-04 22:48:53 UTC (rev 6950) +++ branches/FORMATCHECK/callgrind/callstack.c 2007-10-04 23:12:12 UTC (rev 6951) @@ -275,7 +275,7 @@ BB* bb = jcc->to->bb; if (s>40) s=40; - VG_(printf)("%s> %s(0x%x, 0x%x, ...) [%s / %p]\n", spaces[s%4]+40-s, bb->fn->name, + VG_(printf)("%s> %s(0x%x, 0x%x, ...) [%s / %#lx]\n", spaces[s%4]+40-s, bb->fn->name, pars ? pars[1]:0, pars ? pars[2]:0, bb->obj->name + bb->obj->last_slash_pos, @@ -285,7 +285,7 @@ else if (CLG_(clo).verbose<4) { VG_(printf)("+ %2d ", CLG_(current_call_stack).sp); CLG_(print_short_jcc)(jcc); - VG_(printf)(", SP %p, RA %p\n", sp, ret_addr); + VG_(printf)(", SP %#lx, RA %#lx\n", sp, ret_addr); } else { VG_(printf)(" Pushed "); @@ -365,16 +365,16 @@ if (CLG_(clo).verbose<4) { if (jcc) { /* popped JCC target first */ - VG_(printf)("- %2d %p => ", + VG_(printf)("- %2d %#lx => ", CLG_(current_call_stack).sp, bb_addr(jcc->to->bb)); CLG_(print_addr)(bb_jmpaddr(jcc->from->bb)); - VG_(printf)(", SP %p\n", + VG_(printf)(", SP %#lx\n", CLG_(current_call_stack).entry[CLG_(current_call_stack).sp].sp); CLG_(print_cost)(10, CLG_(sets).full, jcc->cost); } else - VG_(printf)("- %2d [Skipped JCC], SP %p\n", + VG_(printf)("- %2d [Skipped JCC], SP %#lx\n", CLG_(current_call_stack).sp, CLG_(current_call_stack).entry[CLG_(current_call_stack).sp].sp); } @@ -397,7 +397,7 @@ void CLG_(unwind_call_stack)(Addr sp, Int minpops) { Int csp; - CLG_DEBUG(4,"+ unwind_call_stack(sp %p, minpops %d): frame %d\n", + CLG_DEBUG(4,"+ unwind_call_stack(sp %#lx, minpops %d): frame %d\n", sp, minpops, CLG_(current_call_stack).sp); /* We pop old stack frames. Modified: branches/FORMATCHECK/callgrind/debug.c =================================================================== --- branches/FORMATCHECK/callgrind/debug.c 2007-10-04 22:48:53 UTC (rev 6950) +++ branches/FORMATCHECK/callgrind/debug.c 2007-10-04 23:12:12 UTC (rev 6951) @@ -51,7 +51,7 @@ print_indent(s); } - VG_(printf)("BB %p (Obj '%s')", bb_addr(bb), bb->obj->name); + VG_(printf)("BB %#lx (Obj '%s')", bb_addr(bb), bb->obj->name); } static @@ -136,7 +136,7 @@ bb->jmp_offset, bb_jmpaddr(bb)); else #endif - VG_(printf)("%s +%p=%p, ", + VG_(printf)("%s +%#lx=%#lx, ", bb->obj->name + bb->obj->last_slash_pos, bb->offset, bb_addr(bb)); CLG_(print_cxt)(s+8, bbcc->cxt, bbcc->rec_index); @@ -220,7 +220,7 @@ void CLG_(print_short_jcc)(jCC* jcc) { if (jcc) - VG_(printf)("%p => %p [%llu/%llu,%llu,%llu]", + VG_(printf)("%#lx => %#lx [%llu/%llu,%llu,%llu]", bb_jmpaddr(jcc->from->bb), bb_addr(jcc->to->bb), jcc->call_counter, @@ -264,9 +264,9 @@ } ce = CLG_(get_call_entry)(sp); - VG_(printf)("[%-2d] SP %p, RA %p", sp, ce->sp, ce->ret_addr); + VG_(printf)("[%-2d] SP %#lx, RA %#lx", sp, ce->sp, ce->ret_addr); if (ce->nonskipped) - VG_(printf)(" NonSkipped BB %p / %s", + VG_(printf)(" NonSkipped BB %#lx / %s", bb_addr(ce->nonskipped->bb), ce->nonskipped->cxt->fn[0]->name); VG_(printf)("\n"); @@ -381,9 +381,9 @@ CLG_(get_debug_info)(addr, fl_buf, fn_buf, &ln, &si); if (VG_(strcmp)(fn_buf,"???")==0) - VG_(printf)("%p", addr); + VG_(printf)("%#lx", addr); else - VG_(printf)("%p %s", addr, fn_buf); + VG_(printf)("%#lx %s", addr, fn_buf); if (si) { obj_name = VG_(seginfo_filename)(si); Modified: branches/FORMATCHECK/callgrind/dump.c =================================================================== --- branches/FORMATCHECK/callgrind/dump.c 2007-10-04 22:48:53 UTC (rev 6950) +++ branches/FORMATCHECK/callgrind/dump.c 2007-10-04 23:12:12 UTC (rev 6951) @@ -477,7 +477,7 @@ p->addr = addr - bbcc->bb->obj->offset; p->bb_addr = bbcc->bb->offset; - CLG_DEBUG(3, " get_debug_pos(%p): BB %p, fn '%s', file '%s', line %u\n", + CLG_DEBUG(3, " get_debug_pos(%#lx): BB %#lx, fn '%s', file '%s', line %u\n", addr, bb_addr(bbcc->bb), bbcc->cxt->fn[0]->name, p->file->name, p->line); @@ -519,7 +519,7 @@ static void fprint_apos(Int fd, AddrPos* curr, AddrPos* last, file_node* func_file) { CLG_ASSERT(curr->file != 0); - CLG_DEBUG(2, " print_apos(file '%s', line %d, bb %p, addr %p) fnFile '%s'\n", + CLG_DEBUG(2, " print_apos(file '%s', line %d, bb %#lx, addr %#lx) fnFile '%s'\n", curr->file->name, curr->line, curr->bb_addr, curr->addr, func_file->name); @@ -569,7 +569,7 @@ p = VG_(sprintf)(outbuf, "%d ", diff); } else - p = VG_(sprintf)(outbuf, "%p ", curr->addr); + p = VG_(sprintf)(outbuf, "%#lx ", curr->addr); } if (CLG_(clo).dump_bb) { @@ -584,7 +584,7 @@ p += VG_(sprintf)(outbuf+p, "%d ", diff); } else - p += VG_(sprintf)(outbuf+p, "%p ", curr->bb_addr); + p += VG_(sprintf)(outbuf+p, "%#lx ", curr->bb_addr); } if (CLG_(clo).dump_line) { @@ -630,7 +630,7 @@ static void fprint_fcost(Int fd, AddrCost* c, AddrPos* last) { CLG_DEBUGIF(3) { - CLG_DEBUG(2, " print_fcost(file '%s', line %d, bb %p, addr %p):\n", + CLG_DEBUG(2, " print_fcost(file '%s', line %d, bb %#lx, addr %#lx):\n", c->p.file->name, c->p.line, c->p.bb_addr, c->p.addr); CLG_(print_cost)(-5, CLG_(sets).full, c->cost); } @@ -1027,7 +1027,7 @@ CLG_DEBUGIF(8) { for (pm = a; pm < a+n; pm++) { - VG_(printf)(" %3d BB %p, ", pm - qsort_start, + VG_(printf)(" %3d BB %#lx, ", pm - qsort_start, bb_addr((*pm)->bb)); CLG_(print_cxt)(9, (*pm)->cxt, (*pm)->rec_index); } @@ -1081,14 +1081,14 @@ if ((s = a+n-1-pd)>0) { for(r=0;r<s;r++) swap(pc+r, a+n-s+r); } CLG_DEBUGIF(8) { - VG_(printf)(" PV BB %p, ", bb_addr((*pv)->bb)); + VG_(printf)(" PV BB %#lx, ", bb_addr((*pv)->bb)); CLG_(print_cxt)(9, (*pv)->cxt, (*pv)->rec_index); s = pb-pa+1; VG_(printf)(" Lower %d - %d:\n", a-qsort_start, a+s-1-qsort_start); for (r=0;r<s;r++) { pm = a+r; - VG_(printf)(" %3d BB %p, ", + VG_(printf)(" %3d BB %#lx, ", pm-qsort_start,bb_addr((*pm)->bb)); CLG_(print_cxt)(9, (*pm)->cxt, (*pm)->rec_index); } @@ -1098,7 +1098,7 @@ a+n-s-qsort_start, a+n-1-qsort_start); for (r=0;r<s;r++) { pm = a+n-s+r; - VG_(printf)(" %3d BB %p, ", + VG_(printf)(" %3d BB %#lx, ", pm-qsort_start,bb_addr((*pm)->bb)); CLG_(print_cxt)(9, (*pm)->cxt, (*pm)->rec_index); } @@ -1549,7 +1549,7 @@ /* FIXME: Specify Object of BB if different to object of fn */ int i, pos = 0; ULong ecounter = (*p)->ecounter_sum; - pos = VG_(sprintf)(print_buf, "bb=%p ", (*p)->bb->offset); + pos = VG_(sprintf)(print_buf, "bb=%#lx ", (*p)->bb->offset); for(i = 0; i<(*p)->bb->cjmp_count;i++) { pos += VG_(sprintf)(print_buf+pos, "%d %llu ", (*p)->bb->jmp[i].instr, Modified: branches/FORMATCHECK/callgrind/fn.c =================================================================== --- branches/FORMATCHECK/callgrind/fn.c 2007-10-04 22:48:53 UTC (rev 6950) +++ branches/FORMATCHECK/callgrind/fn.c 2007-10-04 23:12:12 UTC (rev 6951) @@ -128,7 +128,7 @@ r = range + 2; found = True; while(r[1]) { - CLG_DEBUG(1, " [%p] Found! Checking %d bytes of [%x %x %x...]\n", + CLG_DEBUG(1, " [%#lx] Found! Checking %d bytes of [%x %x %x...]\n", addr, r[1], code[r[0]], code[r[0]+1], code[r[0]+2]); if (VG_(memcmp)( (void*)(addr+r[0]), code+r[0], r[1]) != 0) { @@ -372,7 +372,7 @@ Char dir[FILENAME_LEN]; UInt line; - CLG_DEBUG(6, " + get_debug_info(%p)\n", instr_addr); + CLG_DEBUG(6, " + get_debug_info(%#lx)\n", instr_addr); if (pSegInfo) { *pSegInfo = VG_(find_seginfo)(instr_addr); @@ -418,7 +418,7 @@ if (line_num) *line_num=0; } - CLG_DEBUG(6, " - get_debug_info(%p): seg '%s', fn %s\n", + CLG_DEBUG(6, " - get_debug_info(%#lx): seg '%s', fn %s\n", instr_addr, !pSegInfo ? (const UChar*)"-" : (*pSegInfo) ? VG_(seginfo_filename)(*pSegInfo) : @@ -445,7 +445,7 @@ /* fn from debug info is idempotent for a BB */ if (bb->fn) return bb->fn; - CLG_DEBUG(3,"+ get_fn_node(BB %p)\n", bb_addr(bb)); + CLG_DEBUG(3,"+ get_fn_node(BB %#lx)\n", bb_addr(bb)); /* get function/file name, line number and object of * the BB according to debug information @@ -458,10 +458,10 @@ /* Use address as found in library */ if (sizeof(Addr) == 4) - p = VG_(sprintf)(fnname, "%08p", bb->offset); + p = VG_(sprintf)(fnname, "%#08lx", bb->offset); else // 64bit address - p = VG_(sprintf)(fnname, "%016p", bb->offset); + p = VG_(sprintf)(fnname, "%#016lx", bb->offset); VG_(sprintf)(fnname+p, "%s", (bb->sect_kind == Vg_SectData) ? " [Data]" : @@ -536,7 +536,7 @@ bb->fn = fn; bb->line = line_num; - CLG_DEBUG(3,"- get_fn_node(BB %p): %s (in %s:%u)\n", + CLG_DEBUG(3,"- get_fn_node(BB %#lx): %s (in %s:%u)\n", bb_addr(bb), fnname, filename, line_num); return fn; Modified: branches/FORMATCHECK/callgrind/main.c =================================================================== --- branches/FORMATCHECK/callgrind/main.c 2007-10-04 22:48:53 UTC (rev 6950) +++ branches/FORMATCHECK/callgrind/main.c 2007-10-04 23:12:12 UTC (rev 6951) @@ -522,12 +522,12 @@ // No instrumentation if it is switched off if (! CLG_(instrument_state)) { - CLG_DEBUG(5, "instrument(BB %p) [Instrumentation OFF]\n", + CLG_DEBUG(5, "instrument(BB %#lx) [Instrumentation OFF]\n", (Addr)closure->readdr); return bbIn; } - CLG_DEBUG(3, "+ instrument(BB %p)\n", (Addr)closure->readdr); + CLG_DEBUG(3, "+ instrument(BB %#lx)\n", (Addr)closure->readdr); /* Set up SB for instrumented IR */ bbOut = deepCopyIRSBExceptStmts(bbIn); @@ -681,7 +681,7 @@ bb->jmpkind = bbIn->jumpkind; } - CLG_DEBUG(3, "- instrument(BB %p): byteLen %u, CJumps %u, CostLen %u\n", + CLG_DEBUG(3, "- instrument(BB %#lx): byteLen %u, CJumps %u, CostLen %u\n", origAddr, bb->instr_len, bb->cjmp_count, bb->cost_count); if (cJumps>0) { CLG_DEBUG(3, " [ "); Modified: branches/FORMATCHECK/callgrind/sim.c =================================================================== --- branches/FORMATCHECK/callgrind/sim.c 2007-10-04 22:48:53 UTC (rev 6950) +++ branches/FORMATCHECK/callgrind/sim.c 2007-10-04 23:12:12 UTC (rev 6951) @@ -790,7 +790,7 @@ UWord *set, tmp_tag; \ UInt use_mask; \ \ - CLG_DEBUG(6,"%s.Acc(Addr %p, size %d): Sets [%d/%d]\n", \ + CLG_DEBUG(6,"%s.Acc(Addr %#lx, size %d): Sets [%d/%d]\n", \ L.name, a, size, set1, set2); \ \ /* First case: word entirely within line. */ \ @@ -808,7 +808,7 @@ idx = (set1 << L.assoc_bits) | (set[0] & ~L.tag_mask); \ L.use[idx].count ++; \ L.use[idx].mask |= use_mask; \ - CLG_DEBUG(6," Hit0 [idx %d] (line %p from %p): %x => %08x, count %d\n",\ + CLG_DEBUG(6," Hit0 [idx %d] (line %#lx from %#lx): %x => %08x, count %d\n",\ idx, L.loaded[idx].memline, L.loaded[idx].iaddr, \ use_mask, L.use[idx].mask, L.use[idx].count); \ return L1_Hit; \ @@ -825,7 +825,7 @@ idx = (set1 << L.assoc_bits) | (tmp_tag & ~L.tag_mask); \ L.use[idx].count ++; \ L.use[idx].mask |= use_mask; \ - CLG_DEBUG(6," Hit%d [idx %d] (line %p from %p): %x => %08x, count %d\n",\ + CLG_DEBUG(6," Hit%d [idx %d] (line %#lx from %#lx): %x => %08x, count %d\n",\ i, idx, L.loaded[idx].memline, L.loaded[idx].iaddr, \ use_mask, L.use[idx].mask, L.use[idx].count); \ return L1_Hit; \ @@ -852,7 +852,7 @@ idx = (set1 << L.assoc_bits) | (set[0] & ~L.tag_mask); \ L.use[idx].count ++; \ L.use[idx].mask |= use_mask; \ - CLG_DEBUG(6," Hit0 [idx %d] (line %p from %p): %x => %08x, count %d\n",\ + CLG_DEBUG(6," Hit0 [idx %d] (line %#lx from %#lx): %x => %08x, count %d\n",\ idx, L.loaded[idx].memline, L.loaded[idx].iaddr, \ use_mask, L.use[idx].mask, L.use[idx].count); \ goto block2; \ @@ -867,7 +867,7 @@ idx = (set1 << L.assoc_bits) | (tmp_tag & ~L.tag_mask); \ L.use[idx].count ++; \ L.use[idx].mask |= use_mask; \ - CLG_DEBUG(6," Hit%d [idx %d] (line %p from %p): %x => %08x, count %d\n",\ + CLG_DEBUG(6," Hit%d [idx %d] (line %#lx from %#lx): %x => %08x, count %d\n",\ i, idx, L.loaded[idx].memline, L.loaded[idx].iaddr, \ use_mask, L.use[idx].mask, L.use[idx].count); \ goto block2; \ @@ -889,7 +889,7 @@ idx = (set2 << L.assoc_bits) | (set[0] & ~L.tag_mask); \ L.use[idx].count ++; \ L.use[idx].mask |= use_mask; \ - CLG_DEBUG(6," Hit0 [idx %d] (line %p from %p): %x => %08x, count %d\n",\ + CLG_DEBUG(6," Hit0 [idx %d] (line %#lx from %#lx): %x => %08x, count %d\n",\ idx, L.loaded[idx].memline, L.loaded[idx].iaddr, \ use_mask, L.use[idx].mask, L.use[idx].count); \ return miss1; \ @@ -904,7 +904,7 @@ idx = (set2 << L.assoc_bits) | (tmp_tag & ~L.tag_mask); \ L.use[idx].count ++; \ L.use[idx].mask |= use_mask; \ - CLG_DEBUG(6," Hit%d [idx %d] (line %p from %p): %x => %08x, count %d\n",\ + CLG_DEBUG(6," Hit%d [idx %d] (line %lx from %#lx): %x => %08x, count %d\n",\ i, idx, L.loaded[idx].memline, L.loaded[idx].iaddr, \ use_mask, L.use[idx].mask, L.use[idx].count); \ return miss1; \ @@ -921,7 +921,7 @@ return (miss1==MemAccess || miss2==MemAccess) ? MemAccess:L2_Hit; \ \ } else { \ - VG_(printf)("addr: %p size: %u sets: %d %d", a, size, set1, set2); \ + VG_(printf)("addr: %#lx size: %u sets: %d %d", a, size, set1, set2); \ VG_(tool_panic)("item straddles more than two cache sets"); \ } \ return 0; \ @@ -952,10 +952,10 @@ line_use* use = &(L2.use[idx]); int i = ((32 - countBits(use->mask)) * L2.line_size)>>5; - CLG_DEBUG(2, " L2.miss [%d]: at %p accessing memline %p\n", + CLG_DEBUG(2, " L2.miss [%d]: at %#lx accessing memline %#lx\n", idx, bb_base + current_ii->instr_offset, memline); if (use->count>0) { - CLG_DEBUG(2, " old: used %d, loss bits %d (%08x) [line %p from %p]\n", + CLG_DEBUG(2, " old: used %d, loss bits %d (%08x) [line %#lx from %#lx]\n", use->count, i, use->mask, loaded->memline, loaded->iaddr); CLG_DEBUG(2, " collect: %d, use_base %p\n", CLG_(current_state).collect, loaded->use_base); @@ -986,13 +986,13 @@ int i, j, idx; UWord tmp_tag; - CLG_DEBUG(6,"L2.Acc(Memline %p): Set %d\n", memline, setNo); + CLG_DEBUG(6,"L2.Acc(Memline %#lx): Set %d\n", memline, setNo); if (tag == (set[0] & L2.tag_mask)) { idx = (setNo << L2.assoc_bits) | (set[0] & ~L2.tag_mask); l1_loaded->dep_use = &(L2.use[idx]); - CLG_DEBUG(6," Hit0 [idx %d] (line %p from %p): => %08x, count %d\n", + CLG_DEBUG(6," Hit0 [idx %d] (line %#lx from %#lx): => %08x, count %d\n", idx, L2.loaded[idx].memline, L2.loaded[idx].iaddr, L2.use[idx].mask, L2.use[idx].count); return L2_Hit; @@ -1007,7 +1007,7 @@ idx = (setNo << L2.assoc_bits) | (tmp_tag & ~L2.tag_mask); l1_loaded->dep_use = &(L2.use[idx]); - CLG_DEBUG(6," Hit%d [idx %d] (line %p from %p): => %08x, count %d\n", + CLG_DEBUG(6," Hit%d [idx %d] (line %#lx from %#lx): => %08x, count %d\n", i, idx, L2.loaded[idx].memline, L2.loaded[idx].iaddr, L2.use[idx].mask, L2.use[idx].count); return L2_Hit; @@ -1040,10 +1040,10 @@ line_use* use = &(cache->use[idx]); \ int c = ((32 - countBits(use->mask)) * cache->line_size)>>5; \ \ - CLG_DEBUG(2, " %s.miss [%d]: at %p accessing memline %p (mask %08x)\n", \ + CLG_DEBUG(2, " %s.miss [%d]: at %#lx accessing memline %#lx (mask %08x)\n", \ cache->name, idx, bb_base + current_ii->instr_offset, memline, mask); \ if (use->count>0) { \ - CLG_DEBUG(2, " old: used %d, loss bits %d (%08x) [line %p from %p]\n",\ + CLG_DEBUG(2, " old: used %d, loss bits %d (%08x) [line %#lx from %#lx]\n",\ use->count, c, use->mask, loaded->memline, loaded->iaddr); \ CLG_DEBUG(2, " collect: %d, use_base %p\n", \ CLG_(current_state).collect, loaded->use_base); \ @@ -1149,7 +1149,7 @@ current_ii = ii; IrRes = (*simulator.I1_Read)(bb_base + ii->instr_offset, ii->instr_size); - CLG_DEBUG(6, "log_1I0D: Ir=%p/%u => Ir %d\n", + CLG_DEBUG(6, "log_1I0D: Ir=%#lx/%u => Ir %d\n", bb_base + ii->instr_offset, ii->instr_size, IrRes); if (CLG_(current_state).collect) { @@ -1177,7 +1177,7 @@ IrRes = (*simulator.I1_Read)(bb_base + ii->instr_offset, ii->instr_size); DrRes = (*simulator.D1_Read)(data, ii->data_size); - CLG_DEBUG(6, "log_1I1Dr: Ir=%p/%u, Dr=%p/%u => Ir %d, Dr %d\n", + CLG_DEBUG(6, "log_1I1Dr: Ir=%#lx/%u, Dr=%#lx/%u => Ir %d, Dr %d\n", bb_base + ii->instr_offset, ii->instr_size, data, ii->data_size, IrRes, DrRes); @@ -1209,7 +1209,7 @@ current_ii = ii; DrRes = (*simulator.D1_Read)(data, ii->data_size); - CLG_DEBUG(6, "log_0I1Dr: Dr=%p/%u => Dr %d\n", + CLG_DEBUG(6, "log_0I1Dr: Dr=%#lx/%u => Dr %d\n", data, ii->data_size, DrRes); if (CLG_(current_state).collect) { @@ -1239,7 +1239,7 @@ IrRes = (*simulator.I1_Read)(bb_base + ii->instr_offset, ii->instr_size); DwRes = (*simulator.D1_Write)(data, ii->data_size); - CLG_DEBUG(6, "log_1I1Dw: Ir=%p/%u, Dw=%p/%u => Ir %d, Dw %d\n", + CLG_DEBUG(6, "log_1I1Dw: Ir=%#lx/%u, Dw=%#lx/%u => Ir %d, Dw %d\n", bb_base + ii->instr_offset, ii->instr_size, data, ii->data_size, IrRes, DwRes); @@ -1270,7 +1270,7 @@ current_ii = ii; DwRes = (*simulator.D1_Write)(data, ii->data_size); - CLG_DEBUG(6, "log_0I1Dw: Dw=%p/%u => Dw %d\n", + CLG_DEBUG(6, "log_0I1Dw: Dw=%#lx/%u => Dw %d\n", data, ii->data_size, DwRes); if (CLG_(current_state).collect) { @@ -1301,7 +1301,7 @@ DwRes = (*simulator.D1_Write)(data2, ii->data_size); CLG_DEBUG(6, - "log_1I2D: Ir=%p/%u, Dr=%p/%u, Dw=%p/%u => Ir %d, Dr %d, Dw %d\n", + "log_1I2D: Ir=%#lx/%u, Dr=%#lx/%u, Dw=%#lx/%u => Ir %d, Dr %d, Dw %d\n", bb_base + ii->instr_offset, ii->instr_size, data1, ii->data_size, data2, ii->data_size, IrRes, DrRes, DwRes); @@ -1338,7 +1338,7 @@ DwRes = (*simulator.D1_Write)(data2, ii->data_size); CLG_DEBUG(6, - "log_0D2D: Dr=%p/%u, Dw=%p/%u => Dr %d, Dw %d\n", + "log_0D2D: Dr=%#lx/%u, Dw=%#lx/%u => Dr %d, Dw %d\n", data1, ii->data_size, data2, ii->data_size, DrRes, DwRes); if (CLG_(current_state).collect) { Modified: branches/FORMATCHECK/memcheck/mc_leakcheck.c =================================================================== --- branches/FORMATCHECK/memcheck/mc_leakcheck.c 2007-10-04 22:48:53 UTC (rev 6950) +++ branches/FORMATCHECK/memcheck/mc_leakcheck.c 2007-10-04 23:12:12 UTC (rev 6951) @@ -65,7 +65,7 @@ void scan_all_valid_memory_catcher ( Int sigNo, Addr addr ) { if (0) - VG_(printf)("OUCH! sig=%d addr=%p\n", sigNo, addr); + VG_(printf)("OUCH! sig=%d addr=%#lx\n", sigNo, addr); if (sigNo == VKI_SIGSEGV || sigNo == VKI_SIGBUS) __builtin_longjmp(memscan_jmpbuf, 1); } @@ -232,7 +232,7 @@ sh_no = find_shadow_for(ptr, lc_shadows, lc_n_shadows); if (VG_DEBUG_LEAKCHECK) - VG_(printf)("ptr=%p -> block %d\n", ptr, sh_no); + VG_(printf)("ptr=%#lx -> block %d\n", ptr, sh_no); if (sh_no == -1) return; @@ -245,7 +245,7 @@ if (lc_markstack[sh_no].state == Unreached) { if (0) - VG_(printf)("pushing %p-%p\n", lc_shadows[sh_no]->data, + VG_(printf)("pushing %#lx-%#lx\n", lc_shadows[sh_no]->data, lc_shadows[sh_no]->data + lc_shadows[sh_no]->szB); tl_assert(lc_markstack[sh_no].next == -1); @@ -257,7 +257,7 @@ if (clique != -1) { if (0) - VG_(printf)("mopup: %d: %p is %d\n", + VG_(printf)("mopup: %d: %#lx is %d\n", sh_no, lc_shadows[sh_no]->data, lc_markstack[sh_no].state); /* An unmarked block - add it to the clique. Add its size to @@ -273,11 +273,11 @@ if (sh_no != clique) { if (VG_DEBUG_CLIQUE) { if (lc_markstack[sh_no].indirect) - VG_(printf)(" clique %d joining clique %d adding %lu+%lu bytes\n", + VG_(printf)(" clique %d joining clique %d adding %u+%lu bytes\n", sh_no, clique, lc_shadows[sh_no]->szB, lc_markstack[sh_no].indirect); else - VG_(printf)(" %d joining %d adding %lu\n", + VG_(printf)(" %d joining %d adding %u\n", sh_no, clique, lc_shadows[sh_no]->szB); } @@ -325,7 +325,7 @@ vki_sigset_t sigmask; if (VG_DEBUG_LEAKCHECK) - VG_(printf)("scan %p-%p\n", start, start+len); + VG_(printf)("scan %#lx-%#lx\n", start, start+len); VG_(sigprocmask)(VKI_SIG_SETMASK, NULL, &sigmask); VG_(set_fault_catcher)(scan_all_valid_memory_catcher); @@ -355,7 +355,7 @@ addr = *(Addr *)ptr; lc_markstack_push_WRK(addr, clique); } else if (0 && VG_DEBUG_LEAKCHECK) - VG_(printf)("%p not valid\n", ptr); + VG_(printf)("%#lx not valid\n", ptr); ptr += sizeof(Addr); } else { /* We need to restore the signal mask, because we were @@ -413,7 +413,7 @@ pass), then the cliques are merged. */ for (i = 0; i < lc_n_shadows; i++) { if (VG_DEBUG_CLIQUE) - VG_(printf)("cliques: %d at %p -> Loss state %d\n", + VG_(printf)("cliques: %d at %#lx -> Loss state %d\n", i, lc_shadows[i]->data, lc_markstack[i].state); if (lc_markstack[i].state != Unreached) continue; @@ -421,7 +421,7 @@ tl_assert(lc_markstack_top == -1); if (VG_DEBUG_CLIQUE) - VG_(printf)("%d: gathering clique %p\n", i, lc_shadows[i]->data); + VG_(printf)("%d: gathering clique %#lx\n", i, lc_shadows[i]->data); lc_markstack_push_WRK(lc_shadows[i]->data, i); @@ -769,7 +769,7 @@ } if (0) - VG_(printf)("ACCEPT %2d %p %p\n", i, seg->start, seg->end); + VG_(printf)("ACCEPT %2d %#lx %#lx\n", i, seg->start, seg->end); lc_scan_memory(seg->start, seg->end+1 - seg->start); } } Modified: branches/FORMATCHECK/memcheck/mc_main.c =================================================================== --- branches/FORMATCHECK/memcheck/mc_main.c 2007-10-04 22:48:53 UTC (rev 6950) +++ branches/FORMATCHECK/memcheck/mc_main.c 2007-10-04 23:12:12 UTC (rev 6951) @@ -2196,7 +2196,7 @@ { tl_assert(sizeof(UWord) == sizeof(SizeT)); if (0) - VG_(printf)("helperc_MAKE_STACK_UNINIT %p %lu\n", base, len ); + VG_(printf)("helperc_MAKE_STACK_UNINIT %#lx %lu\n", base, len ); # if 0 /* Really slow version */ ------------------------------------------------------------------------- This SF.net email is sponsored by: Splunk Inc. Still grepping through log files to find problems? Stop. Now Search log events and configuration files using AJAX and a browser. Download your FREE copy of Splunk now >> http://get.splunk.com/ _______________________________________________ Valgrind-developers mailing list Valgrind-developers@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/valgrind-developers