Commit 89bb70e9 by Jamie Madill

Mark TF Feedback array element capture as unsupported.

According to dEQP this is an undefined section of the spec. Mark it as unsupported for now until we can investgate if it's possible on certain platforms. Noticed this when testing with the Program refactor. BUG=angleproject:1123 TEST=dEQP-GLES3.functional.transform_feedback.array_element.* Change-Id: I830dddf418e28fb4536f09500f4069def1d97b58 Reviewed-on: https://chromium-review.googlesource.com/295984Reviewed-by: 's avatarCorentin Wallez <cwallez@chromium.org> Reviewed-by: 's avatarGeoff Lang <geofflang@chromium.org> Tested-by: 's avatarJamie Madill <jmadill@chromium.org>
parent aa40feea
...@@ -1625,6 +1625,13 @@ bool Program::linkValidateTransformFeedback(InfoLog &infoLog, ...@@ -1625,6 +1625,13 @@ bool Program::linkValidateTransformFeedback(InfoLog &infoLog,
} }
} }
// TODO(jmadill): investigate if we can support capturing array elements.
if (tfVaryingName.find('[') != std::string::npos)
{
infoLog << "Capture of array elements not currently supported.";
return false;
}
// All transform feedback varyings are expected to exist since packVaryings checks for them. // All transform feedback varyings are expected to exist since packVaryings checks for them.
ASSERT(found); ASSERT(found);
UNUSED_ASSERTION_VARIABLE(found); UNUSED_ASSERTION_VARIABLE(found);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment