RFR: C1 cleanup

[Roman Kennke]

Am Donnerstag, den 08.12.2016, 12:05 +0100 schrieb Aleksey Shipilev:
> On 12/08/2016 12:00 PM, Roman Kennke wrote:
> > http://cr.openjdk.java.net/~rkennke/c1-cleanup/webrev.00/
> 
> *) Shouldn't this assert be on lir_cas_long branch only? Or is it in
> upstream
> that (odd) way?
> 
> 1600 void LIR_Assembler::emit_compare_and_swap(LIR_OpCompareAndSwap*
> op) {
> 1601   assert(VM_Version::supports_cx8(), "wrong machine");

It's in upstream like this.

> *) Please break this line:
> 1461   LIR_OpShenandoahWriteBarrier(LIR_Opr obj, LIR_Opr result,
> CodeEmitInfo*
> info, bool need_null_check) : LIR_Op1(lir_shenandoah_wb, obj, result,
> T_OBJECT,
> lir_patch_none, info), _need_null_check(need_null_check) {
> 
> Otherwise looks good.

Ok, pushed with that line broken in half.

Roman