RFR: 8282355: compiler/arguments/TestCodeEntryAlignment.java failed "guarantee(sect->end() <= tend) failed: sanity"
Dean Long
dlong at openjdk.java.net
Sun Mar 13 03:19:45 UTC 2022
On Sat, 12 Mar 2022 23:41:42 GMT, Jie Fu <jiefu at openjdk.org> wrote:
>> This change adds extra stub space for large values of CodeEntryAlignment, and it changes the test to try large values of CodeEntryAlignment.
>
> test/hotspot/jtreg/compiler/arguments/TestCodeEntryAlignment.java line 69:
>
>> 67:
>> 68: public static void driver() throws IOException {
>> 69: for (int align = 32; align <= 1024; align *= 2) {
>
> This wouldn't test for `-XX:CodeEntryAlignment=16`.
> However, we used to find a bug with `-XX:CodeEntryAlignment=16`.
> https://github.com/openjdk/jdk/pull/7485
>
> Why not testing with `-XX:CodeEntryAlignment=16`?
The only reason is because -XX:CodeCacheSegmentSize=16 gives an error. If it's important to test for -XX:CodeEntryAlignment=16 then I'll have to rework the logic to be more clever. I'll probably have to use WhiteBox APIs to read the default CodeCacheSegmentSize and only set it to values that are >= the default. It's unfortunate that -XX:CodeEntryAlignment doesn't adjust the default value of CodeCacheSegmentSize automatically.
-------------
PR: https://git.openjdk.java.net/jdk/pull/7800
More information about the hotspot-compiler-dev
mailing list