[trans-mem] RFC Fix missing REG_TM notes
Patrick Marlier
patrick.marlier@gmail.com
Wed Jan 11 09:09:00 GMT 2012
On 01/10/2012 05:39 PM, Richard Henderson wrote:
> Also outstanding is that the return value setup for beginTransaction
> needs to be adjusted into the new block. I.e. we current generate
>
> (set (reg eax) (call _ITM_beginTransaction))
> (set (reg psuedo) (reg eax))
> .Lrestart:
> // rest of tm block
>
> This must be
>
> (set (reg eax) (call _ITM_beginTransaction))
> .Lrestart:
> (set (reg psuedo) (reg eax))
> // rest of tm block
>
> lest the register allocator do the wrong thing.
Yep good point. It means moving the insn from one BB to the next BB but
I have no idea how to do it and where it should be done.
Thanks.
--
Patrick.
More information about the Gcc
mailing list