RFR: 8282355: compiler/arguments/TestCodeEntryAlignment.java failed "guarantee(sect->end() <= tend) failed: sanity" [v2]
Dean Long
dlong at openjdk.java.net
Mon Mar 14 08:34:40 UTC 2022
On Mon, 14 Mar 2022 00:15:20 GMT, Jie Fu <jiefu at openjdk.org> wrote:
>> Dean Long has updated the pull request incrementally with one additional commit since the last revision:
>>
>> improvement based on review
>
> src/hotspot/share/runtime/stubRoutines.cpp line 219:
>
>> 217: TraceTime timer("StubRoutines generation 1", TRACETIME_LOG(Info, startuptime));
>> 218: // Add extra space for large CodeEntryAlignment
>> 219: int max_stubs = 10;
>
> Thanks for your update.
>
> There are about 30 stubs (max number of stubs) to be generated in `generate_initial()` for x86_64.
> Why not `max_stubs = 30` ?
I instrumented the align() call in my private build to count how many used align(CodeEntryAlignment), and I counted only 7, then I rounded up to 10. Maybe I should change the name to max_aligned_stubs to make it more clear?
-------------
PR: https://git.openjdk.java.net/jdk/pull/7800
More information about the hotspot-compiler-dev
mailing list