[lilliput-jdk17u:lilliput] Integrated: 8308172: [Lilliput/JDK17] Revert disabling vzeroupper
Roman Kennke
rkennke at openjdk.org
Tue May 16 12:00:15 UTC 2023
On Tue, 16 May 2023 10:57:52 GMT, Roman Kennke <rkennke at openjdk.org> wrote:
> While looking through the diff between Lilliput/JDK17 and upstream JDK17, I noticed one change in library_call.cpp that should not be there, it probably slipped through from debugging.
>
> void clear_upper_avx() {
> #ifdef X86
> - if (UseAVX >= 2) {
> + if (false && UseAVX >= 2) {
> C->set_clear_upper_avx(true);
> }
> #endif
> }
>
> This prevents emitting vzeroupper instructions on x86_64. I believe this will have performance impact when transitioning between AVX and SSE code. We should revert this.
This pull request has now been integrated.
Changeset: a7a8efbd
Author: Roman Kennke <rkennke at openjdk.org>
URL: https://git.openjdk.org/lilliput-jdk17u/commit/a7a8efbdaa7daad48ae2a04eaf44fb70d27f8baf
Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
8308172: [Lilliput/JDK17] Revert disabling vzeroupper
Reviewed-by: shade
-------------
PR: https://git.openjdk.org/lilliput-jdk17u/pull/18
More information about the lilliput-dev
mailing list