RFR: 8254175: Build no-pch configuration in debug mode for submit checks [v2]

Robin Westberg rwestberg at openjdk.java.net
Thu Oct 8 07:21:57 UTC 2020


On Thu, 8 Oct 2020 07:19:21 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

>> no-pch configuration is supposed to expose missing include dependencies. But currently it runs with default (release)
>> bits, which misses symbols hidden in debug code. We should consider building it in debug mode.
>> Attention @rwestberg.
>> 
>> Testing:
>>   - [x] GH workflow still works, see the builds in the [latest run](https://github.com/shipilev/jdk/actions/runs/293802758)
>
> Aleksey Shipilev has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Drop "debug" from the names

Looks good to me! Just a similar comment as in #546, the name is a bit too long now, so perhaps just drop the "debug"
from the description?

Looks good!

.github/workflows/submit.yml line 108:

> 106:           - build release
> 107:           - build debug
> 108:           - build debug hotspot no-pch

Suggestion:

          - build hotspot no-pch

.github/workflows/submit.yml line 113:

> 111:             flags: --enable-debug
> 112:             artifact: -debug
> 113:           - flavor: build debug hotspot no-pch

Suggestion:

          - flavor: build hotspot no-pch

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

Marked as reviewed by rwestberg (Committer).

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



More information about the build-dev mailing list