i965: Delete some unnecessary code in brw_report_shader_time().
authorKenneth Graunke <kenneth@whitecape.org>
Wed, 15 Apr 2015 09:04:17 +0000 (02:04 -0700)
committerKenneth Graunke <kenneth@whitecape.org>
Fri, 17 Apr 2015 19:07:33 +0000 (12:07 -0700)
It is true that a gl_shader_program with ID 0 will be a fixed-function
fragment program; a gl_program with ID 0 but NULL gl_shader_program
means that it's a fixed-function vertex shader.

But that's not terribly interesting or relevant to what we're doing.
We just need to know that ID 0 means "fixed function".

Signed-off-by: Kenneth Graunke <kenneth@whitecape.org>
Reviewed-by: Jason Ekstrand <jason.ekstrand@intel.com>
Reviewed-by: Anuj Phogat <anuj.phogat@gmail.com>
src/mesa/drivers/dri/i965/brw_program.c

index 0ca63de0cf9c4599a1494834071c6d5c9f8c4a29..7ea08e614735dd0ea2dab9981da2145d2c79fc88 100644 (file)
@@ -447,14 +447,9 @@ brw_report_shader_time(struct brw_context *brw)
       if (prog) {
          shader_num = prog->Name;
 
-         /* The fixed function fragment shader generates GLSL IR with a Name
-          * of 0, and nothing else does.
-          */
          if (prog->Label) {
             shader_name = prog->Label;
-         } else if (shader_num == 0 &&
-             (brw->shader_time.types[i] == ST_FS8 ||
-              brw->shader_time.types[i] == ST_FS16)) {
+         } else if (shader_num == 0) {
             shader_name = "ff";
          } else {
             shader_name = "glsl";