RFR: 8255615: Zero: demote ZeroStack::abi_stack_available guarantee to assert

AlphaHot github.com+58410019+alphahot at openjdk.java.net
Fri Oct 30 08:00:42 UTC 2020


On Thu, 29 Oct 2020 19:13:06 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

> It is currently guarantee(), which slows down release bits unnecessarily:
> 
> inline int ZeroStack::abi_stack_available(Thread *thread) const {
>   guarantee(Thread::current() == thread, "should run in the same thread");
> 
> On my TR 3970X, changing that `guarantee` to `assert` improves:
>  - Zero x86_64 release "make images" times from ~9.5 minutes to ~7.5 minutes
>  - Zero x86_64 release "make bootcycle-images" times from ~49 minutes to ~44 minutes
> 
> Attention @jerboaa and @gnu-andrew, who have to suffer through the Zero build times occasionally.

Marked as reviewed by AlphaHot at github.com (no known OpenJDK username).

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

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


More information about the hotspot-runtime-dev mailing list