]> gcc.gnu.org Git - gcc.git/commit
This patch fixes PR bootstrap/57077.
authorTeresa Johnson <tejohnson@google.com>
Mon, 29 Apr 2013 13:22:46 +0000 (13:22 +0000)
committerTeresa Johnson <tejohnson@gcc.gnu.org>
Mon, 29 Apr 2013 13:22:46 +0000 (13:22 +0000)
commitf41f80f90846d26a89ed5a6440bc283b745235ac
tree9daf54908866e15052bfcfc011bc4753b4b363fb
parent315bbd2e3c15dea3528259be2aee2876dec33843
This patch fixes PR bootstrap/57077.

This patch fixes PR bootstrap/57077. Certain new uses of apply_probability
are actually scaling the counts up, and the scale factor should not
be treated as a probability as the value may exceed REG_BR_PROB_BASE.
One example (from the PR) is when scaling counts up in LTO when merging
profiles. Another example I found when preparing the patch to use
the rounding divide in more places is when inlining COMDAT functions.

Add new helper function apply_scale that does the scaling without
the probability range check. I audited the new uses of apply_probability
and changed the calls as appropriate.

2013-04-29  Teresa Johnson  <tejohnson@google.com>

PR bootstrap/57077
* basic-block.h (apply_scale): New function.
(apply_probability): Use apply_scale.
* gimple-streamer-in.c (input_bb): Ditto.
* lto-streamer-in.c (input_cfg): Ditto.
* lto-cgraph.c (merge_profile_summaries): Ditto.
* tree-optimize.c (execute_fixup_cfg): Ditto.
* tree-inline.c (copy_bb): Update comment to use
apply_scale.
(copy_edges_for_bb): Ditto.
(copy_cfg_body): Ditto.

From-SVN: r198416
gcc/ChangeLog
gcc/basic-block.h
gcc/gimple-streamer-in.c
gcc/lto-cgraph.c
gcc/lto-streamer-in.c
gcc/tree-inline.c
gcc/tree-optimize.c
This page took 0.061133 seconds and 5 git commands to generate.