On Fri, Sep 27, 2013 at 4:08 PM, Xinliang David Li wrote:
> d.growth -= (info->size
> * (100 - PARAM_VALUE (PARAM_COMDAT_SHARING_PROBABILITY))
> + 50) / 100;
>
> What is the purpose of '50' here?
Rounding after division by 100.
> The patch is fine for Google branch.
>
> Other tunings to
d.growth -= (info->size
* (100 - PARAM_VALUE (PARAM_COMDAT_SHARING_PROBABILITY))
+ 50) / 100;
What is the purpose of '50' here?
The patch is fine for Google branch.
Other tunings to think about -- I think the sharing probability should
not be a fixed value -- but depending on the functio
This patch increases comdat-sharing-probability to 80 under -Os. This
reduces the amount of inlining and helps internal benchmarks.
Unfortunately, this causes slight regression on spec 2006. Ok for
google branches if all tests pass?
- Easwaran
comdat_sharing.patch
Description: Binary data
