On Thu, 8 Oct 2020 07:12:12 GMT, Aleksey Shipilev <sh...@openjdk.org> wrote:

>> That makes sense. We should change the order in #547 then first? So the 
>> final thing would be:
>> 
>> - build release
>> - build debug
>> - build hotspot no-pch debug
>> - build hotspot zero no-pch debug
>> - build hotspot minimal no-pch debug
>
> ...or we drop `debug` to altogether:
> 
> - build release
> - build debug
> - build hotspot no-pch
> - build hotspot zero no-pch
> - build hotspot minimal no-pch

Yeah I had the same comment there. :) Perhaps just drop "debug" qualifier as 
they all have it. And perhaps drop the
no-pch from the two new ones, it's IMO not that important to display it in the 
description (unless we were to add for
example zero without no-pch) - the full config is fairly easy to figure out 
from the yml or the test log anyway.

-------------

PR: https://git.openjdk.java.net/jdk/pull/546

Reply via email to