RFR(XS): 8220660: [s390]: debug build broken after JDK-8220301

[Doerr, Martin]

Hi Thomas,

no problem. I just pushed it after "good and trivial" from Aleksey.

Best regards,
Martin


-----Original Message-----
From: Thomas Schatzl <thomas.schatzl at oracle.com> 
Sent: Donnerstag, 14. März 2019 12:30
To: Doerr, Martin <martin.doerr at sap.com>; hotspot-runtime-dev at openjdk.java.net; Lindenmaier, Goetz <goetz.lindenmaier at sap.com>; Volker Simonis (volker.simonis at gmail.com) <volker.simonis at gmail.com>
Subject: Re: RFR(XS): 8220660: [s390]: debug build broken after JDK-8220301

Hi,

On Thu, 2019-03-14 at 10:36 +0000, Doerr, Martin wrote:
> Hi,
> 
> I'd like to fix an s390 debug build error introduced by JDK-8220301:
> src/hotspot/cpu/s390/gc/g1/g1BarrierSetAssembler_s390.cpp: In member
> function 'void
> G1BarrierSetAssembler::g1_write_barrier_post(MacroAssembler*,
> DecoratorSet, Register, Register, Register, Register, Register)':
> /usr/work/d056149/openjdk/jdk/src/hotspot/cpu/s390/gc/g1/g1BarrierSet
> Assembler_s390.cpp:319:57: error: comparison is always true due to
> limited range of data type [-Werror=type-limits]
>    assert((unsigned int)G1CardTable::dirty_card_val() <= 255,
> "otherwise check this code");
> 
> This assertion is pointless since dirty_card_val is uint8_t.
> 
> Please review the removal of it:
> http://cr.openjdk.java.net/~mdoerr/8220660_fix_s390_build/webrev.00/
> 
> Best regards,
> Martin
> 

  good and trivial. Sorry for the breakage.

Thanks,
  Thomas