RFR 8219242 [lworld] C1 aastore on a Phi node needs flattened array check
    Tobias Hartmann 
    tobias.hartmann at oracle.com
       
    Thu Feb 21 10:05:59 UTC 2019
    
    
  
Hi Ioi,
On 21.02.19 05:17, Ioi Lam wrote:
> http://cr.openjdk.java.net/~iklam/valhalla/8219242-aastore-phi-node-needs-flat-check.v02/
> 
> Note that I changed this line
> 
> 1687   bool obj_store = x->elt_type() == T_OBJECT; assert(x->elt_type() != T_ARRAY, "never used");
> 
> because x->elt_type() is determine by the bytecode (Xastore) and no bytecode will set it to T_ARRAY.
Looks good but please move the assert into a new line before pushing.
Best regards,
Tobias
    
    
More information about the valhalla-dev
mailing list