On Thu, 11 Sep 2025 21:05:57 GMT, Erik Joelsson <er...@openjdk.org> wrote:
>> SendaoYan has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Create a seperate macro FLAGS_SETUP_SVE > > make/autoconf/flags-other.m4 line 119: > >> 117: CHECK_AVAILABLE: [ >> 118: # Check whether the compiler supports the Arm C Language >> Extensions (ACLE) >> 119: # for SVE. Set SVE_CFLAGS to -march=armv8-a+sve if it does. > > This comment is a bit out of date with the code now. This is just the check > available part. Maybe move this to before the macro call? Thanks, the description comment has been moved to before the macro called. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/27073#discussion_r2342770993