Commit eaee472d by Jan Hubicka Committed by Jan Hubicka

profile-count.h (apply_probability, [...]): Fix checks for zero.


	* profile-count.h (apply_probability,
	apply_scale, probability_in): Fix checks for zero.

From-SVN: r249563
parent c016fd32
2017-06-22 Jan Hubicka <hubicka@ucw.cz>
* profile-count.h (apply_probability,
apply_scale, probability_in): Fix checks for zero.
2017-06-22 Rainer Orth <ro@CeBiTec.Uni-Bielefeld.DE> 2017-06-22 Rainer Orth <ro@CeBiTec.Uni-Bielefeld.DE>
* incpath.c (add_sysroot_to_chain): Allow for $SYSROOT prefix. * incpath.c (add_sysroot_to_chain): Allow for $SYSROOT prefix.
......
...@@ -255,7 +255,7 @@ public: ...@@ -255,7 +255,7 @@ public:
profile_count apply_probability (int prob) const profile_count apply_probability (int prob) const
{ {
gcc_checking_assert (prob >= 0 && prob <= REG_BR_PROB_BASE); gcc_checking_assert (prob >= 0 && prob <= REG_BR_PROB_BASE);
if (*this == profile_count::zero ()) if (m_val == 0)
return *this; return *this;
if (!initialized_p ()) if (!initialized_p ())
return profile_count::uninitialized (); return profile_count::uninitialized ();
...@@ -267,24 +267,25 @@ public: ...@@ -267,24 +267,25 @@ public:
/* Return *THIS * NUM / DEN. */ /* Return *THIS * NUM / DEN. */
profile_count apply_scale (int64_t num, int64_t den) const profile_count apply_scale (int64_t num, int64_t den) const
{ {
if (*this == profile_count::zero ()) if (m_val == 0)
return *this; return *this;
if (!initialized_p ()) if (!initialized_p ())
return profile_count::uninitialized (); return profile_count::uninitialized ();
profile_count ret; profile_count ret;
gcc_checking_assert (num >= 0 && den > 0);
/* FIXME: shrink wrapping violates this sanity check. */ /* FIXME: shrink wrapping violates this sanity check. */
gcc_checking_assert ((num >= 0 gcc_checking_assert ((num <= REG_BR_PROB_BASE
&& (num <= REG_BR_PROB_BASE || den <= REG_BR_PROB_BASE) || 1);
|| den <= REG_BR_PROB_BASE)
&& den > 0) || 1);
ret.m_val = RDIV (m_val * num, den); ret.m_val = RDIV (m_val * num, den);
ret.m_quality = MIN (m_quality, count_adjusted); ret.m_quality = MIN (m_quality, count_adjusted);
return ret; return ret;
} }
profile_count apply_scale (profile_count num, profile_count den) const profile_count apply_scale (profile_count num, profile_count den) const
{ {
if (*this == profile_count::zero () || num == profile_count::zero ()) if (m_val == 0)
return profile_count::zero (); return *this;
if (num.m_val == 0)
return num;
if (!initialized_p () || !num.initialized_p () || !den.initialized_p ()) if (!initialized_p () || !num.initialized_p () || !den.initialized_p ())
return profile_count::uninitialized (); return profile_count::uninitialized ();
gcc_checking_assert (den > 0); gcc_checking_assert (den > 0);
...@@ -306,7 +307,7 @@ public: ...@@ -306,7 +307,7 @@ public:
OVERALL. */ OVERALL. */
int probability_in (profile_count overall) int probability_in (profile_count overall)
{ {
if (*this == profile_count::zero ()) if (!m_val)
return 0; return 0;
if (!initialized_p () || !overall.initialized_p ()) if (!initialized_p () || !overall.initialized_p ())
return REG_BR_PROB_BASE / 2; return REG_BR_PROB_BASE / 2;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment