RFR(XS): 8211207: AArch64: Fix build failure after JDK-8211029

Pengfei Li (Arm Technology China) Pengfei.Li at arm.com
Thu Sep 27 08:08:20 UTC 2018


Hi Aleksey,

Thanks for your review. Please see new patch at http://cr.openjdk.java.net/~njian/8211207/webrev.01/
I've added line breaks after "default:" and restored a missing break in switch-case.

Could you help push this new patch if no other issues?

--
Thanks,
Pengfei


> -----Original Message-----
> 
> 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



More information about the build-dev mailing list