Release store in C2 putfield

Andrew Haley aph at redhat.com
Thu Sep 4 08:21:46 UTC 2014


On 04/09/14 09:14, Volker Simonis wrote:
> I've just checked that we only use this flag (i.e.
> as_Store()->is_unordered()/as_Store()->is_release()) on our Itanium
> port. Other platforms use the MemBarRelease/MemBarStoreStore nodes
> which are there anyway. On Itanium, we don't use these nodes that's
> why we need the flag.
> 
> But I don't see how this bothers anybody. If you don't use this flag
> in your .ad file in your Store nodes it will have no side effect. Or
> am I missing something?

I think you must be.  We generate a release store if the node asks for
one.  AArch64 doesn't usually need separate barriers.

But that's not the point, really: the point is that not every reference
store needs to be a release, which is how it is right now.

Andrew.



More information about the hotspot-dev mailing list