commit a9ca8e2025368aaf613915cc12e7e1bd1aa4befe
author Geoff Lang <geofflang@chromium.org> Thu Oct 22 14:34:20 2015 +0000
committer Geoff Lang <geofflang@chromium.org> Thu Oct 22 14:34:31 2015 +0000
tree f4aa7e87bf0d909f5f98ea3e6cb1300903d5dc24
parent a9c5288192f90ebf03b0d26d86964b4fbce74bcb

Revert "Implement TransformFeedbackGL."

Failing on AMD Linux bots.

This reverts commit a9c5288192f90ebf03b0d26d86964b4fbce74bcb.

Change-Id: I2a2db8c3d96c4b927d7d784486c6df772ae63084
Reviewed-on: https://chromium-review.googlesource.com/308130
Reviewed-by: Geoff Lang <geofflang@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

src/libANGLE/Program.cpp

diff --git a/src/libANGLE/Program.cpp b/src/libANGLE/Program.cpp
index 40d694f..814deb9 100644
--- a/src/libANGLE/Program.cpp
+++ b/src/libANGLE/Program.cpp
@@ -235,7 +235,7 @@
 Program::Data::Data()
     : mAttachedFragmentShader(nullptr),
       mAttachedVertexShader(nullptr),
-      mTransformFeedbackBufferMode(GL_INTERLEAVED_ATTRIBS)
+      mTransformFeedbackBufferMode(GL_NONE)
 {
 }
 
@@ -2014,12 +2014,6 @@
                 }
                 uniqueNames.insert(tfVaryingName);
 
-                if (varying->isArray())
-                {
-                    infoLog << "Capture of arrays is undefined and not supported.";
-                    return false;
-                }
-
                 // TODO(jmadill): Investigate implementation limits on D3D11
                 size_t componentCount = gl::VariableComponentCount(varying->type);
                 if (mData.mTransformFeedbackBufferMode == GL_SEPARATE_ATTRIBS &&
@@ -2037,9 +2031,10 @@
             }
         }
 
+        // TODO(jmadill): investigate if we can support capturing array elements.
         if (tfVaryingName.find('[') != std::string::npos)
         {
-            infoLog << "Capture of array elements is undefined and not supported.";
+            infoLog << "Capture of array elements not currently supported.";
             return false;
         }