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

[Aleksey Shipilev]

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. The affected method is private, and so what that `assert` really does is checks that no internal Zero code calls it with a wrong `thread` -- something that should be caught by `fastdebug` builds.
> 
> 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.

This pull request has now been integrated.

Changeset: 9a0cf587
Author:    Aleksey Shipilev <shade at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/9a0cf587
Stats:     1 line in 1 file changed: 0 ins; 0 del; 1 mod

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

Reviewed-by: sgehwolf

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

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