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