[PATCH, i386] RTM support

Kirill Yukhin kirill.yukhin@gmail.com
Mon Mar 5 11:31:00 GMT 2012


Hello Uros,

> As the first remark, you don't have to add BLKmode memory clobbers.
> unspec_volatile RTX is considered to use and clobber all memory:
>

Thanks, fixed!

>
> But, I think that we want to use code label from the top of the false
> branch instead of ".+6". The question is, how to get it ...

Agreed, this seems not as nice, but still it works :)
I still do not understand, why not to put something like this?
"xbegin\t1f\n1:"
This is local label, which is not intercept others...

>
> Please use ix86_expand_setcc instead of opencoding sete insn.
>

Fixed!

Anything else?

Thanks, K



More information about the Gcc-patches mailing list