https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110852
--- Comment #14 from Jan Hubicka <hubicka at ucw dot cz> --- > I thought the goal was to handle what is in predict-18.c, i.e. > b * __builtin_expect (c, 0) > or similar. If it is about > __builtin_expect_with_probability (b, 42, 0.25) * > __builtin_expect_with_probability (c, 0, 0.42) > sure, my version will merge the probabilities, while you'll pick the > probability from > the 0 case. Probability from 0 case is better estimate, so I think it makes sense to handle it right. I did not take that much stats on how often it happens, but on my TODO list is to turn this into value range predictor which may have better chance of success. We can also handle other constants than INTEGER_CST. I will see if I can clean up the code bit more or add a comment, since it is indeed bit confusing as written now. Will also look into more testcases. Thanks a lot! Honza
