Silence overactive assert in ipa-inline
Jan Hubicka
hubicka@ucw.cz
Fri Nov 17 19:53:00 GMT 2017
Hi,
with frequencies not being capped by 100 it is easy to run into roundoff errors
that are more than 1. Maybe I will need to give up on this assert (which would
be pity as it is useful) but for now I just made it bit more tolerant.
Bootstrapped/regtested x86_64-linux.
Honza
* ipa-fnsummary.c (estimate_node_size_and_time): Be more tolerant for
roundoff errors.
Index: ipa-fnsummary.c
===================================================================
--- ipa-fnsummary.c (revision 254884)
+++ ipa-fnsummary.c (working copy)
@@ -2744,7 +2744,7 @@ estimate_node_size_and_time (struct cgra
gcc_checking_assert (time >= 0);
/* nonspecialized_time should be always bigger than specialized time.
Roundoff issues however may get into the way. */
- gcc_checking_assert ((nonspecialized_time - time) >= -1);
+ gcc_checking_assert ((nonspecialized_time - time * 0.99) >= -1);
/* Roundoff issues may make specialized time bigger than nonspecialized
time. We do not really want that to happen because some heurstics
More information about the Gcc-patches
mailing list