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

Add uniform buffer subtests exposing ANGLE bugs #2481

Merged
merged 2 commits into from Aug 15, 2017

Conversation

Oletus
Copy link
Contributor

@Oletus Oletus commented Aug 14, 2017

These tests pass on ANGLE's GL backend but failed on the DX backend
until very recent fixes.

These tests pass on ANGLE's GL backend but failed on the DX backend
until very recent fixes.
@Oletus Oletus force-pushed the structs-in-interface-blocks-test branch from 54a66b1 to 19bff4d Compare August 14, 2017 13:34
@zhenyao
Copy link
Contributor

zhenyao commented Aug 14, 2017

LGTM with a request for linking to the ANGLE bug.
Feel free to merge yourself once it's added.

wrapper_t UBOStruct;
};

// This is intended to reproduce a specific ANGLE bug that triggers when the wrapper struct is passed to a function.
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you add a reference link to the ANGLE bug here?

color_t UBOColors[2];
};

// This is intended to reproduce a specific ANGLE bug that triggers when a struct from an array of structs in an interface block is passed to a function.
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you add a reference link to the ANGLE bug here?

@Oletus
Copy link
Contributor Author

Oletus commented Aug 15, 2017

Added the bug links, merging. Thanks for the reviews!

@Oletus Oletus merged commit dec40ec into KhronosGroup:master Aug 15, 2017
@Oletus Oletus deleted the structs-in-interface-blocks-test branch August 15, 2017 07:00
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.

None yet

2 participants