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