On Fri, 20 Sep 2024 12:34:38 GMT, Erik Joelsson <er...@openjdk.org> wrote:
>> 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. Modified, Thank you! ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/21083#discussion_r1770886456
