Messages in this thread |  | | | Date | Wed, 22 Feb 2012 13:58:10 -0500 | | From | Jason Baron <> | | Subject | Re: [PATCH 00/10] jump label: introduce very_[un]likely + cleanups + docs |
| |
On Wed, Feb 22, 2012 at 09:14:19AM -0800, Richard Henderson wrote: > On 02/22/12 07:32, Peter Zijlstra wrote: > > So I clicked the link Jason provided in his 10/10 Documentation patch > > and stumbled upon: > > > > http://gcc.gnu.org/ml/gcc-patches/2009-07/msg01558.html > > > > Where rth suggests that __attribute__((hot,cold)) might work on the > > destination labels. Trying this my compiler (4.6.1+crap) pukes all over > > me suggesting this isn't (yet) implemented. > > > > Richard, is something like that still on the table? > > It's still a possibility. I gave Jason a patch for that quite some time > ago; I don't recall hearing whether it turned out to actually be useful. > > > r~
I don't think I ever quite got it working, and I've unfortunately misplaced it at this point. I am definitely interested in trying it again though. If you can re-send it, I'll try it.
Also, I think I seem to recall it wouldn't help the -Os case, b/c we don't get block re-ordering?
Thanks,
-Jason
|  |