RFR(S): 8087341: C2 doesn't optimize redundant memory operations with G1

[Aleksey Shipilev]

On 01/28/2016 06:04 PM, Aleksey Shipilev wrote:
> On 01/28/2016 03:49 PM, Roland Westrelin wrote:
>> http://cr.openjdk.java.net/~roland/8087341/webrev.00/
> 
> This looks good, for most tests here:
>   http://cr.openjdk.java.net/~shade/8087341/G1BackToBackStores.java
> 
> The generated code indeed shows commoned loads/stores with this patch,
> and some other things that can be improved in the codegen -- I'll file
> the separate issue(s) for that. I think this one is better to be renamed
> to something more specific, e.g. "Overly wide StoreLoad barrier in G1
> breaks load/store coalescing"?

Found an even more convincing example:
 http://cr.openjdk.java.net/~shade/8087341/G1LoopStores.java

Happy to re-run once Roland has the patch with Vladimir's comments.

Cheers,
-Aleksey




-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20160129/dd664618/signature-0001.asc>