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