Skip to content

Conversation

Daylight-Peng
Copy link
Contributor

Correct glGetUniformSubroutineuiv return value comparison logic

glGetUniformSubroutineuiv returns the value of the subroutine uniform at
location .
The return value means the "subroutine index", not the "uniform location",
so it can not compare the return value with location value.

Affects:
KHR-GL40.shader_subroutine.subroutine_uniform_reset

Daylight-Peng and others added 2 commits November 2, 2023 16:49
glGetUniformSubroutineuiv returns the value of the subroutine uniform at
location <location>.
The return value means the "subroutine index", not the "uniform location",
so it can not compare the return value with location value.

Affects:
KHR-GL40.shader_subroutine.subroutine_uniform_reset

Signed-off-by: Riliang Peng <[email protected]>
@Daylight-Peng
Copy link
Contributor Author

Please help to review, thanks.

@Daylight-Peng
Copy link
Contributor Author

Could you please help to check?

@lordalcol
Copy link
Contributor

Hi @Daylight-Peng, sorry for the delay. We are currently reviewing and testing this change in Khronos CI.

@Daylight-Peng
Copy link
Contributor Author

Hi @Daylight-Peng, sorry for the delay. We are currently reviewing and testing this change in Khronos CI.

Is it OK now?

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@lordalcol
Copy link
Contributor

Internal Khronos CL: https://gerrit.khronos.org/c/vk-gl-cts/+/17894

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.

3 participants