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

Achieve full test coverage of TransformationAccessChain #4286

Closed
afd opened this issue May 22, 2021 · 1 comment · Fixed by #4304
Closed

Achieve full test coverage of TransformationAccessChain #4286

afd opened this issue May 22, 2021 · 1 comment · Fixed by #4304
Labels
component:fuzzer Relates to the spirv-fuzz tool

Comments

@afd
Copy link
Contributor

afd commented May 22, 2021

The regular spirv-fuzz tests miss some statements in this transformation class, and it would be good to cover them.
@afd afd added the component:fuzzer Relates to the spirv-fuzz tool label May 22, 2021
@sliu-UIUC
Copy link
Contributor

I'll take care of this issue.

@sliu-UIUC sliu-UIUC mentioned this issue May 26, 2021
Merged
@afd afd closed this as completed in #4304 Jun 2, 2021
afd pushed a commit that referenced this issue Jun 2, 2021
@sliu-UIUC
spirv-fuzz: add tests for full coverage of TransformationAccessChain (#…
26cdce9 
…4304)

Fixes #4286 by achieving full coverage of the transformation.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
component:fuzzer Relates to the spirv-fuzz tool
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

spirv-fuzz: add tests for full coverage of TransformationAccessChain sliu-UIUC/SPIRV-Tools
2 participants
@afd @sliu-UIUC