-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 On 08/18/11 15:59, Richard Henderson wrote: > On 08/17/2011 12:21 AM, Richard Guenther wrote: >> The patch itself looks sensible, though I am surprised ifcvt >> doesn't run in cfglayout mode (so you have to use reg notes to >> find probabilities ...) > > It does run in cfglayout mode. > > Jeff, I believe you're supposed to get the probabilities from some > combination of > > bb->frequency edge->probability EDGE_FREQUENCY(edge) OK. Here's the revised patch. There's other places in ifcvt.c that utilize the notes that I didn't modify. Bootstrapped & regression tested x86_64-unknown-linux-gnu. Also verified performance data hasn't changed materially. OK for trunk? Thanks, jeff -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.11 (GNU/Linux) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/ iQEcBAEBAgAGBQJOgPzuAAoJEBRtltQi2kC7uQkH+wTyLl3MwKkqC11ozyOAVSQ9 CrlwLAuN7JV7kZcguKZ9GpXYHePZHZHCISXaVv3LQQBXnE7PehJOWsd1D5BQRv2/ eqVHIAOYg0LamY2cRiWW8pKMiMjs7vb9q0fiehQGg0zxAJMc9crBjwLPGFjZAksw UNzzon/NKfSMYsz9X/olDfk8DPa1DmAjBnNOcHzKLGdx7KDa6Npo20k3D/PwDbIe y1Ff9pZBXJP6tNU+0cn9lyyt+w6ghFQRkpKJoJ6iSOxKQ6v23+03o4sT5GHv3Gvy 07R6NJU9vqt7a9GvxcyJ9BsOlCCJ/pA/4lProHrdcAZrOYWZAA4uoJeDBPOEq+g= =a5LI -----END PGP SIGNATURE-----