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

[msan] Add more NEON VST tests #100189

Merged
merged 3 commits into from
Jul 23, 2024
Merged

Conversation

thurstond
Copy link
Contributor

This includes additional test cases that are currently unsupported by MSan: VST_{2,3,4} with floating-point parameters, and VST1x_{2,3,4}.

This includes additional test cases that are currently unsupported by
MSan: VST_{2,3,4} with floating-point parameters, and VST1x_{2,3,4}.
@thurstond thurstond requested a review from vitalybuka July 23, 2024 20:07
@thurstond thurstond merged commit e858a92 into llvm:main Jul 23, 2024
4 of 5 checks passed
thurstond added a commit to thurstond/llvm-project that referenced this pull request Jul 23, 2024
thurstond added a commit that referenced this pull request Jul 23, 2024
thurstond added a commit to thurstond/llvm-project that referenced this pull request Jul 24, 2024
…meters

Cloning the vst_ functions to apply them to the shadows did not work if the arguments were floating-point, since the shadows are integers. This patch changes MSan to create an intrinsic of the correct integer types.

Additionally, this patch adds support for vst1x_{2,3,4}; these can be handled similarly to vst_{2,3,4}, since in all cases we are cloning/adapting
the corresponding function.

This also updates and enables the test introduced in llvm#100189
thurstond added a commit to thurstond/llvm-project that referenced this pull request Jul 25, 2024
…meters

Cloning the vst_ functions to apply them to the shadows did not work if the arguments were floating-point, since the shadows are integers. This patch changes MSan to create an intrinsic of the correct integer types.

Additionally, this patch adds support for vst1x_{2,3,4}; these can be handled similarly to vst_{2,3,4}, since in all cases we are cloning/adapting the corresponding function.

This also updates and enables the test introduced in llvm#100189
thurstond added a commit to thurstond/llvm-project that referenced this pull request Jul 25, 2024
…meters

Cloning the vst_ functions to apply them to the shadows did not work if the arguments were floating-point, since the shadows are integers. This patch changes MSan to create an intrinsic of the correct integer types.

Additionally, this patch adds support for vst1x_{2,3,4}; these can be handled similarly to vst_{2,3,4}, since in all cases we are cloning/adapting the corresponding function.

This also updates and enables the test introduced in llvm#100189
yuxuanchen1997 pushed a commit that referenced this pull request Jul 25, 2024
Summary:
This includes additional test cases that are currently unsupported by
MSan: VST_{2,3,4} with floating-point parameters, and VST1x_{2,3,4}.

Test Plan: 

Reviewers: 

Subscribers: 

Tasks: 

Tags: 


Differential Revision: https://phabricator.intern.facebook.com/D60251111
yuxuanchen1997 pushed a commit that referenced this pull request Jul 25, 2024
vitalybuka pushed a commit to thurstond/llvm-project that referenced this pull request Jul 29, 2024
…meters

Cloning the vst_ functions to apply them to the shadows did not work if the arguments were floating-point, since the shadows are integers. This patch changes MSan to create an intrinsic of the correct integer types.

Additionally, this patch adds support for vst1x_{2,3,4}; these can be handled similarly to vst_{2,3,4}, since in all cases we are cloning/adapting the corresponding function.

This also updates and enables the test introduced in llvm#100189
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