Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Backport] 8262355: Support for AVX-512 opmask register allocation. #639

Closed

Conversation

JinZhonghui
Copy link
Collaborator

Summary: 8262355: Support for AVX-512 opmask register allocation.

Test Plan: ci jtreg

Reviewed-by: JoshuaZhuwj

Issue: #607

@JinZhonghui
Copy link
Collaborator Author

@JoshuaZhuwj Please help to review this.

Copy link
Collaborator

@JoshuaZhuwj JoshuaZhuwj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Zhonghui, Thanks for your contributing.

As our offline communication, VectorMaskGen, LoadVectorMasked and StoreVectorMasked should be introduced since they are the basis of mask support.

See my other comments inline.

src/hotspot/cpu/x86/assembler_x86.cpp Show resolved Hide resolved
src/hotspot/cpu/x86/assembler_x86.hpp Show resolved Hide resolved
src/hotspot/cpu/x86/x86_64.ad Outdated Show resolved Hide resolved
src/hotspot/cpu/x86/x86_64.ad Show resolved Hide resolved
src/hotspot/cpu/x86/x86_64.ad Show resolved Hide resolved
src/hotspot/cpu/x86/x86_64.ad Outdated Show resolved Hide resolved
src/hotspot/cpu/x86/x86_64.ad Show resolved Hide resolved
src/hotspot/cpu/x86/x86_32.ad Show resolved Hide resolved
src/hotspot/cpu/x86/gc/z/zBarrierSetAssembler_x86.cpp Outdated Show resolved Hide resolved
…tion

Summary: 8261553: Efficient mask generation using BMI2 BZHI instruction
         Only introduce genmask() and bzhiq()

Test Plan: ci jtreg

Reviewed-by: JoshuaZhuwj

Issue: dragonwell-project#690
@JinZhonghui JinZhonghui force-pushed the JDK-8262355 branch 5 times, most recently from 2aa9519 to c85ad35 Compare October 27, 2023 04:19
Summary: 8262355: Support for AVX-512 opmask register allocation.
	Add VectorMaskGen, LoadVectorMasked and StoreVectorMasked related code.

Test Plan: ci jtreg

Reviewed-by: JoshuaZhuwj

Issue: dragonwell-project#607
@JoshuaZhuwj
Copy link
Collaborator

Just for future reference.
See the new version of "AVX-512 opmask register allocation" at #703

@JinZhonghui JinZhonghui closed this Nov 2, 2023
@JinZhonghui JinZhonghui reopened this Nov 2, 2023
@JinZhonghui JinZhonghui closed this Nov 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants