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