More on memory barriers

Doug Lea dl at cs.oswego.edu
Wed Sep 17 11:16:36 UTC 2014


On 09/17/2014 06:31 AM, Andrew Haley wrote:
> On 09/16/2014 08:40 AM, Andrew Haley wrote:
>> On 15/09/14 17:20, Vitaly Davidovich wrote:
>>> Looking at hg history, MemBarStoreStore was added a few years ago, whereas
>>> the code in question is much older.  The comments in the changelist adding
>>> MemBarStoreStore seem to indicate it was done to address a specific issue,
>>> and my guess is that it wasn't "retrofitted" into all possible places.
>>
>> That sounds plausible.  I'll change this to a StoreStore in the AArch64
>> port and do some testing.
>
> Bah, that doesn't work.  Escape analysis assumes that a StoreStore
> is only used in certain contexts.  Back to the drawing board.
>

The setup for StoreStore seems suspicious. I believe that this could
only work in C2 if done in the way I mentioned: StoreStore must be
handled identically to Release by c2, but possibly more cheaply
matched. Can StoreStore be reworked as a subtype or property of
Release?

-Doug




More information about the hotspot-dev mailing list