RFR(S): Clean-up BarrierSetC2

Tobias Hartmann tobias.hartmann at oracle.com
Wed Dec 11 08:04:07 UTC 2019


Hi Nils,

nice cleanup!

Why did you change "Node* var" -> "Node *var" in LoadNode::split_through_phi?

Thanks,
Tobias

On 10.12.19 15:00, Nils Eliasson wrote:
> Hi,
> 
> I can across a lot of dead code in BarrierSetC2 while debugging a completely separate problem. This
> is leftovers from legacy barrier implementations that no longer is needed.
> 
> This is a quick cleanup where non-used parts are removed. I tried to keep it as simple as possible
> to avoid introducing any bugs. There are additional cleanups that can be done in these files later.
> 
> Bug: https://bugs.openjdk.java.net/browse/JDK-8235653
> 
> Webrev: http://cr.openjdk.java.net/~neliasso/8235653/webrev.01
> 
> 
> Please review,
> 
> Nils Eliasson
> 


More information about the hotspot-compiler-dev mailing list