RFR: 8378470: Add future Intel family number
David Holmes
dholmes at openjdk.org
Wed Feb 25 11:01:03 UTC 2026
On Tue, 24 Feb 2026 18:04:54 GMT, Volodymyr Paprotski <vpaprotski at openjdk.org> wrote:
>> src/hotspot/cpu/x86/stubGenerator_x86_64.cpp line 4204:
>>
>>> 4202:
>>> 4203: #ifdef COMPILER2
>>> 4204: if ((UseAVX >= 2) && EnableX86ECoreOpts) {
>>
>> Can you explain this change please?
>
> Nova Lake will have hybrid cores, so trying to hook indexof into just the EnableX86ECoreOpts option.
>
> I debated putting it into a separate PR? I think it should had been dependent on just the EnableX86ECoreOpts option to begin with. On hybrid cores however, it becomes a necessity though.
But hybrid core support is not new, so should this have been changed when we first added hybrid support?
I would suggest a separate JBS issue and PR to make the change clearer. Thanks.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/29852#discussion_r2850478398
More information about the hotspot-dev
mailing list