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.

Sign up for GitHub

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

Jump to bottom

[JIT] Add embedded broadcast support to AVX512BW_BroadcastScalarToVector512 #86986

Open
Ruihan-Yin opened this issue May 31, 2023 · 2 comments
Open

[JIT] Add embedded broadcast support to AVX512BW_BroadcastScalarToVector512 #86986

Ruihan-Yin opened this issue May 31, 2023 · 2 comments
Assignees
@tannergooding @Ruihan-Yin
Labels
area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI avx512 Related to the AVX-512 architecture
Milestone
Future

Comments

@Ruihan-Yin
Copy link
Contributor

This issue is the follow-up for the embedded broadcast enabling PR,

Try to make AVX512BW_BroadcastScalarToVector512 node can be contained by embedded broadcast compatible node to handle the possible use cases like:

Avx512F.Add(vector1, Avx512BW.BroadcastScalarToVector512(Vector128.CreateScalarUnsafe((byte)5)).AsInt32())
@dotnet-issue-labeler dotnet-issue-labeler bot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label May 31, 2023
@ghost ghost added the untriaged New issue has not been triaged by the area owner label May 31, 2023
@Ruihan-Yin Ruihan-Yin mentioned this issue May 31, 2023
Merged
@tannergooding tannergooding added area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI avx512 Related to the AVX-512 architecture and removed needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners labels Jun 1, 2023
@ghost
Copy link

ghost commented Jun 1, 2023

Tagging subscribers to this area: @JulieLeeMSFT, @jakobbotsch
See info in area-owners.md if you want to be subscribed.

Issue Details

This issue is the follow-up for the embedded broadcast enabling PR,

Try to make AVX512BW_BroadcastScalarToVector512 node can be contained by embedded broadcast compatible node to handle the possible use cases like:

Avx512F.Add(vector1, Avx512BW.BroadcastScalarToVector512(Vector128.CreateScalarUnsafe((byte)5)).AsInt32())
Author: Ruihan-Yin
Assignees: -
Labels:

area-CodeGen-coreclr, untriaged, arch-avx512
Milestone: -

@JulieLeeMSFT JulieLeeMSFT assigned Ruihan-Yin and unassigned Ruihan-Yin Jun 2, 2023
@JulieLeeMSFT JulieLeeMSFT removed the untriaged New issue has not been triaged by the area owner label Jun 2, 2023
@JulieLeeMSFT JulieLeeMSFT added this to the 8.0.0 milestone Jun 2, 2023
@tannergooding tannergooding modified the milestones: 8.0.0, Future Aug 8, 2023
@tannergooding
Copy link
Member

Won't get to this for .NET 8 and its not a core scenario needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tannergooding tannergooding

@Ruihan-Yin Ruihan-Yin

Labels
area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI avx512 Related to the AVX-512 architecture
Projects
None yet
Milestone
Future
Development

No branches or pull requests
3 participants
@tannergooding @JulieLeeMSFT @Ruihan-Yin