On Thu, 2009-06-18 at 03:17 -0700, David Brownell wrote:
> On Wednesday 17 June 2009, Zach Welch wrote:
> > --- src/target/trace.h (working copy)
> > +++ src/target/trace.h (working copy)
> > @@ -28,7 +28,7 @@
> > typedef struct trace_point_s
> > {
> > uint32_t address;
> > - u64 hit_counter;
> > + uint64_t hit_counter;
> > } trace_point_t;
> >
> > typedef struct trace_s
>
> On x86_64 builds I now get:
> gcc -std=gnu99 -DHAVE_CONFIG_H -I. -I../.. -I../../src/helper
> -I../../src/jtag -I../../src/xsvf -I/usr/local/include -Wall
> -Wstrict-prototypes -Wformat-security -Wextra -Wno-unused-parameter
> -Wbad-function-cast -Wcast-align -Wredundant-decls -Werror -MT trace.lo -MD
> -MP -MF .deps/trace.Tpo -c trace.c -o trace.o
> cc1: warnings being treated as errors
> trace.c: In function ‘handle_trace_point_command’:
> trace.c:63: warning: format ‘%lld’ expects type ‘long long int’, but argument
> 4 has type ‘uint64_t’
> make[3]: *** [trace.lo] Error 1
> make[3]: *** Waiting for unfinished jobs....
>
> Making it "%llu" doesn't help. I guess this is one of the ways
> that the previous "u64" behaved differently from "uint64_t"...
I found this myself and just committed a patch to fix it, but I simply
use a cast. There probably is a better way, but this fixes the build
for now. Fortunately, this is the only variable in the tree that uses
the uint64_t type, so the consequences should be minor.
Cheers,
Zach
_______________________________________________
Openocd-development mailing list
Openocd-development@lists.berlios.de
https://lists.berlios.de/mailman/listinfo/openocd-development
