[aarch64-port-dev ] RFR: 8135018: AARCH64: Missing memory barriers for CMS collector

[Dean Long]

On 9/3/2015 11:30 AM, Andrew Haley wrote:
> On 09/03/2015 06:52 PM, Dean Long wrote:
>
>> I think you should include the GC alias too.  I thought earlier
>> discussions decided that a barrier wasn't needed.
> I don't remember exactly that.  There was some talk about alternative
> algorithms, but I don't think it's come to anything yet.  There was a
> long discussion about conditional marking, and the StoreLoad barrier
> went in.

I may be remembering an internal discussion.  Let's see what the GC 
folks say.

dl

>> In LIRGenerator::do_StoreIndexed, it looks like the barrier is being
>> done twice, first in store_check, and again in post_barrier. Doing
>> it in post_barrier makes more sense to me than doing it in
>> store_check.
> Okay, I'll check.
>
> Thanks,
> Andrew.