[11] RFR (XXS): 8206476: Wrong assert in phase_enum_2_phase_string() in referenceProcessorPhaseTimes.cpp

Thomas Schatzl thomas.schatzl at oracle.com
Mon Jul 9 14:58:55 UTC 2018


Hi,

On Mon, 2018-07-09 at 10:54 -0400, Kim Barrett wrote:
> > On Jul 9, 2018, at 6:13 AM, Thomas Schatzl <thomas.schatzl at oracle.c
> > om> wrote:
> > […]
> > Actually there is an existing (correct) macro for the whole assert.
> > Replaced that line with the macro as follows:
> > 
> > […]
> > CR:
> > https://bugs.openjdk.java.net/browse/JDK-8206476
> > Webrev:
> > http://cr.openjdk.java.net/~tschatzl/8206476/webrev/index.html
> > Testing:
> > hs-tier1-3,jdk-tier1
> > 
> > Thanks,
> >  Thomas
> 
> Looks good.
> 
> Thanks for spotting and using the existing helper macro.
> 

  thanks for your review.

Thomas



More information about the hotspot-gc-dev mailing list