On Fri, Dec 9, 2011 at 10:58 AM, Kai Tietz <[email protected]> wrote: > 2011/12/9 Richard Guenther <[email protected]>: >> On Fri, Dec 9, 2011 at 9:51 AM, Kai Tietz <[email protected]> wrote: >>> Hi, >>> >>> this patch fixes for windows native target print-formatter used about >>> long-long type. >>> >>> ChangeLog >>> >>> 2011-12-09 Kai Tietz <[email protected]> >>> >>> * ira-color.c (print_hard_regs_subforest): Use >>> HOST_WIDEST_INT_PRINT_DEC instead of %lld. >>> >>> Tested for i686-w64-mingw32, x86_64-w64-mingw32, and >>> x86_64-unknown-linux-gnu. Ok for apply? >> >> Hm? struct allocno_hard_regs uses a long long int cost member, >> so why is %lld wrong? If it doesn't work then you should change >> the cost member to use HOST_WIDEST_INT as well I guess >> (and verify all (indirect) uses). >> >> Richard. > > Issue is that the printf-formatter %ll doesn't necessarily is present > for windows native targets. For these targets the formatter is %I64 > here instead.
We seem to have HOST_LONG_LONG_FORMAT, why not use that? > Kai
