On Sat, Sep 24, 2011 at 11:24 AM, Richard Guenther
<richard.guent...@gmail.com> wrote:
> On Thu, Sep 15, 2011 at 6:26 PM, Paolo Bonzini <bonz...@gnu.org> wrote:
>> On 09/15/2011 06:19 PM, Richard Henderson wrote:
>>>
>>> I wouldn't go that far.  They *used* to be compiler barriers,
>>> but clearly something broke at some point without anyone noticing.
>>> We don't know how many versions are affected until we debug it.
>>> For all we know it broke in 4.5 and 4.4 is fine.
>>
>> 4.4 is not necessarily fine, it may also be that an unrelated 4.5 change
>> exposed a latent bug.
>>
>> But indeed Richard Sandiford mentioned offlist that perhaps
>> ALIAS_SET_MEMORY_BARRIER machinery broke.  Fixing the bug in 4.5/4.6/4.7
>> will definitely shed more light.
>
> ALIAS_SET_MEMORY_BARRIER?!  Eh, well - that will make TBAA consider
> something a memory barrier (I suppose the value of the macro is just zero),
> but clearly non-TBAA alias analysis will happily disambiguate things.

Nope, it's implemented/tested in a way that should work (on RTL, that is).

Richard.

Reply via email to