RFR: 8313530: VM build without C2 fails after JDK-8312579 [v2]

[Hao Sun]

On Wed, 23 Aug 2023 09:11:49 GMT, Gergö Barany <gbarany at openjdk.org> wrote:

>> The EnableVectorSupport flag is declared in `opto/c2_globals.hpp`, which is not included if `COMPILER2` is not set. But after my changes for [JDK-8312579](https://bugs.openjdk.org/browse/JDK-8312579) we try to access this flag in some places guarded by `#if COMPILER2_OR_JVMCI`.
>> 
>> This PR moves some flags from `c2_globals.hpp` to the shared `compiler_globals.hpp`, so that they are accessible even if C2 is disabled but JVMCI is enabled.
>
> Gergö Barany has updated the pull request incrementally with two additional commits since the last revision:
> 
>  - Add copies of Vector API flags in jvmci_globals.hpp
>  - Revert "8313530: VM build without C2 fails after JDK-8312579"
>    
>    This reverts commit d82e89c469e91f78f9c2e5b28c725b0e1ba0fb8c.

> > Verified with Linux/AArch64 and Linux/x86_64 that VM build without C2 is passed now.
> 
> The debug build still fails: #15419 . So I assume the verified builds were all release builds, right?

Sorry for the late reply.
Yes. I only verified the release builds.

And thanks for your fix in https://github.com/openjdk/jdk/pull/15419 👍

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

PR Comment: https://git.openjdk.org/jdk/pull/15384#issuecomment-1694775166