On Thu, 19 Sep 2024 16:35:14 GMT, Hamlin Li <m...@openjdk.org> wrote:
>> Sorry, I had to remind myself of how this works. We actually set this as a >> separate parameter on the Setup macro: `OPTIMIZATION := HIGH` > > Thanks. I'm sorry too, I'm not familiar with the build system. > What you expected could be something like below? > > diff --git a/make/modules/jdk.incubator.vector/Lib.gmk > b/make/modules/jdk.incubator.vector/Lib.gmk > index 5e52277919a..c6c6103a301 100644 > --- a/make/modules/jdk.incubator.vector/Lib.gmk > +++ b/make/modules/jdk.incubator.vector/Lib.gmk > @@ -41,11 +41,12 @@ endif > ifeq ($(call isTargetOs, linux)+$(call isTargetCpu, > riscv64)+$(INCLUDE_COMPILER2), true+true+true) > $(eval $(call SetupJdkLibrary, BUILD_LIBSLEEF, \ > NAME := sleef, \ > + OPTIMIZATION := HIGH, \ > SRC := libsleef/lib, \ > EXTRA_SRC := libsleef/generated, \ > DISABLED_WARNINGS_gcc := unused-function sign-compare > tautological-compare ignored-qualifiers, \ > DISABLED_WARNINGS_clang := unused-function sign-compare > tautological-compare ignored-qualifiers, \ > - CFLAGS := $(CFLAGS_JDKLIB) -O3 -march=rv64gcv, \ > + CFLAGS := $(CFLAGS_JDKLIB) -march=rv64gcv, \ > LDFLAGS := $(LDFLAGS_JDKLIB) \ > $(call SET_SHARED_LIBRARY_ORIGIN), \ > LIBS := $(JDKLIB_LIBS) \ Yes, exactly. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/21083#discussion_r1768546861