From: hubicka Date: Sun, 23 Oct 2011 14:30:24 +0000 (+0000) Subject: * ipa-inline.c (estimate_badness): Scale up and handle overflows. X-Git-Url: http://git.sourceforge.jp/view?a=commitdiff_plain;ds=sidebyside;h=48976c0798a3541bd92888e554e41ae1654c4453;p=pf3gnuchains%2Fgcc-fork.git * ipa-inline.c (estimate_badness): Scale up and handle overflows. git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@180336 138bc75d-0d04-0410-961f-82ee72b054a4 --- diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 82105b41510..2602f76c41f 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,7 @@ +2011-10-23 Jan Hubicka + + * ipa-inline.c (estimate_badness): Scale up and handle overflows. + 2011-10-23 Uros Bizjak PR target/50788 diff --git a/gcc/ipa-inline.c b/gcc/ipa-inline.c index f53f00174a8..662c6b3cb5d 100644 --- a/gcc/ipa-inline.c +++ b/gcc/ipa-inline.c @@ -822,8 +822,10 @@ edge_badness (struct cgraph_edge *edge, bool dump) /* Result must be integer in range 0...INT_MAX. Set the base of fixed point calculation so we don't lose much of precision for small bandesses (those are interesting) yet we don't - overflow for growths that are still in interesting range. */ - badness = ((gcov_type)growth) * (1<<18); + overflow for growths that are still in interesting range. + + Fixed point arithmetic with point at 8th bit. */ + badness = ((gcov_type)growth) * (1<<(19+8)); badness = (badness + div / 2) / div; /* Overall growth of inlining all calls of function matters: we want to @@ -838,10 +840,14 @@ edge_badness (struct cgraph_edge *edge, bool dump) We might mix the valud into the fraction by taking into account relative growth of the unit, but for now just add the number into resulting fraction. */ + if (badness > INT_MAX / 2) + { + badness = INT_MAX / 2; + if (dump) + fprintf (dump_file, "Badness overflow\n"); + } growth_for_all = estimate_growth (callee); badness += growth_for_all; - if (badness > INT_MAX - 1) - badness = INT_MAX - 1; if (dump) { fprintf (dump_file,