dojump.c
35.2 KB
-
This patch fixes an issue where expansion of an ORIF expression arbitrarily... · 99206ca9
This patch fixes an issue where expansion of an ORIF expression arbitrarily applied the probability that the entire condition was true to just the first condition. When the ORIF true probability was 100%, this resulted in the second condition's jump being given a count of zero (since the first condition's jump got 100% of the count), leading to incorrect function splitting when it had a non-zero probability in reality. Since there currently isn't better information about which condition resulted in the ORIF being true, apply a 50-50 probability that it is the first vs. second condition that caused the entire expression to be true, so that neither condition's true label ends up as a 0-count bb. An equivalent fix is made for ANDIF expansion. 2013-10-02 Teresa Johnson <tejohnson@google.com> * dojump.c (do_jump_1): Divide probability between both conditions of a TRUTH_ORIF_EXPR. From-SVN: r203126
Teresa Johnson committed