On 09/27/2018 09:10 AM, Pengfei Li (Arm Technology China) wrote:
> Please help review this patch that fixed these warnings in AArch64 backend.
> webrev: http://cr.openjdk.java.net/~njian/8211207/webrev.00/

Please break the line after "default:" to match the style in 
c1_LIRGenerator_aarch64.cpp here:

 795         default: break;
 796       }
 797     }
 798     default: break;
 799   }

Otherwise looks good and trivial.

Thanks,
-Aleksey

Reply via email to