> Hi, > Can you help to review this patch? > Thanks > > This is a continuation of work based on [1] by @XiaohongGong, most work was > done in that pr. In this new pr, just rebased the code in [1], then added > some minor changes (renaming of calling method, add libsleef as extra lib in > CI cross-build on aarch64 in github workflow); I aslo tested the combination > of following scenarios: > * at build time > * with/without sleef > * with/without sve support > * at runtime > * with/without sleef > * with/without sve support > > [1] https://github.com/openjdk/jdk/pull/16234 > > ## Regression Test > * test/jdk/jdk/incubator/vector/ > * test/hotspot/jtreg/compiler/vectorapi/ > > ## Performance Test > Previously, @XiaohongGong has shared the data: > https://github.com/openjdk/jdk/pull/16234#issuecomment-1767727028
Hamlin Li has updated the pull request incrementally with two additional commits since the last revision: - rename - resolve magicus's comments ------------- Changes: - all: https://git.openjdk.org/jdk/pull/18294/files - new: https://git.openjdk.org/jdk/pull/18294/files/668d08ce..7b7ec312 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=18294&range=02 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=18294&range=01-02 Stats: 63 lines in 7 files changed: 0 ins; 39 del; 24 mod Patch: https://git.openjdk.org/jdk/pull/18294.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/18294/head:pull/18294 PR: https://git.openjdk.org/jdk/pull/18294