On 09/04/18 16:16, Tapani Pälli wrote:
One nit that you can consider below ..

Reviewed-by: Tapani Pälli <tapani.pa...@intel.com>

On 09.04.2018 14:00, Lionel Landwerlin wrote:
Not all of our platforms support GLSL ES 3.0 so extend the coverage by
lowering the GLSL requirements.

Signed-off-by: Lionel Landwerlin <lionel.g.landwer...@intel.com>
---
  tests/spec/intel_blackhole_render/blackhole_draw.c | 9 +++++----
  1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/tests/spec/intel_blackhole_render/blackhole_draw.c b/tests/spec/intel_blackhole_render/blackhole_draw.c
index 32249c8a7..cbabffc0c 100644
--- a/tests/spec/intel_blackhole_render/blackhole_draw.c
+++ b/tests/spec/intel_blackhole_render/blackhole_draw.c
@@ -54,10 +54,11 @@ piglit_display(void)
      prog = piglit_build_simple_program(
  #if defined(PIGLIT_USE_OPENGL)
          "#version 330\n"
+        "in vec4 piglit_vertex;\n"
  #elif defined(PIGLIT_USE_OPENGL_ES2) || defined(PIGLIT_USE_OPENGL_ES3)
-        "#version 300 es\n"
+        "#version 100\n"
+        "attribute vec4 piglit_vertex;\n"
  #endif
-        "in vec4 piglit_vertex;\n"
          "void main()\n"
          "{\n"
          "  gl_Position = piglit_vertex;\n"
@@ -66,8 +67,8 @@ piglit_display(void)
          "#version 330\n"
          "out vec4 color;\n"
  #elif defined(PIGLIT_USE_OPENGL_ES2) || defined(PIGLIT_USE_OPENGL_ES3)
-        "#version 300 es\n"
-        "out highp vec4 color;\n"
+        "#version 100\n"
+        "#define color gl_FragColor\n"

Maybe just use gl_FragColor below?

Sure, thanks!


  #endif
          "void main()\n"
          "{\n"



_______________________________________________
Piglit mailing list
Piglit@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/piglit

Reply via email to