RFR: 8352675: Support Intel AVX10 converged vector ISA feature detection [v10]

Jatin Bhateja jbhateja at openjdk.org
Sat May 3 07:54:46 UTC 2025


On Sat, 3 May 2025 07:41:43 GMT, Vladimir Ivanov <vlivanov at openjdk.org> wrote:

> Ok, thanks! I wasn't sure you finished the pass.
> 
> I'm still seeing dynamic memory allocation which IMO unnecessarily complicates the implementation. Bitmap size is fixed and well-known at compile time. It enables `VM_Feature` class to embed the array of proper size inline. And it eliminates all the problems related to undesired sharing of backed array. (Also, `pre_initialize()` is not needed as well.)

pre_initialize was put in place because codeCache_init () proceeds VM_Version_init() and  it makes calls to some assembler routines which checks for existinace of certain targets features. Its an ordering issue, pre_initialize simply allocates feature vector  upfront to prevent crashing.

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

PR Comment: https://git.openjdk.org/jdk/pull/24329#issuecomment-2848492777


More information about the graal-dev mailing list