RFR (M) CR 8050147: StoreLoad barrier interferes with stack usages

[Vladimir Kozlov]

Method declaration should be done in src/share/vm/runtime/os.hpp
You need make changes in closed sources too.

Thanks,
Vladimir

On 9/3/14 11:03 AM, Aleksey Shipilev wrote:
> On 09/03/2014 09:40 PM, John Rose wrote:
>> On Sep 3, 2014, at 3:21 AM, Aleksey Shipilev <aleksey.shipilev at oracle.com> wrote:
>>
>>> On 08/29/2014 07:45 PM, Aleksey Shipilev wrote:
>>>> There you go, please review:
>>>>   http://cr.openjdk.java.net/~shade/8050147/webrev.05/
>>>>
>>>> This webrev passes the full JPRT smoke tests on all platforms (trivial
>>>> changes in closed code are required), and the performance improvement on
>>>> microbenchmarks is still there.
>>>
>>> (sorry for bugging again)
>>>
>>> Any opinions?
>>
>> :-)  I like that better, even though it makes tiny touches to a bunch of files.  They are the right touches.
>
> Thanks for review, John!
>
> I think I need a second Reviewer. Anyone? :)
>
> Thanks,
> -Aleksey.
>