On Fri, Dec 9, 2011 at 9:51 AM, Kai Tietz <ktiet...@googlemail.com> wrote:
> Hi,
>
> this patch fixes for windows native target print-formatter used about
> long-long type.
>
> ChangeLog
>
> 2011-12-09 Kai Tietz <kti...@redhat.com>
>
> * 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.
> Kai
>
> Index: ira-color.c
> ===================================================================
> --- ira-color.c (revision 182092)
> +++ ira-color.c (working copy)
> @@ -498,7 +498,8 @@
> fprintf (f, " ");
> fprintf (f, "%d:(", node->preorder_num);
> print_hard_reg_set (f, node->hard_regs->set, false);
> - fprintf (f, ")@%lld\n", node->hard_regs->cost);
> + fprintf (f, ")@" HOST_WIDEST_INT_PRINT_DEC "\n",
> + (HOST_WIDEST_INT) node->hard_regs->cost);
> print_hard_regs_subforest (f, node->first, level + 1);
> }
> }
