gallivm: cleanup the code (found by coverity)
authorZack Rusin <zackr@vmware.com>
Tue, 30 Mar 2010 12:55:17 +0000 (08:55 -0400)
committerZack Rusin <zackr@vmware.com>
Tue, 30 Mar 2010 12:55:17 +0000 (08:55 -0400)
the condition can't be false, declerations are ok even if we don't
emit any.

src/gallium/auxiliary/gallivm/lp_bld_tgsi_soa.c

index 95cd6a0ecf3eccb58649486ff31aad0afae77d6c..8901e656aed290c4f527c1663d960300349a36b8 100644 (file)
@@ -666,30 +666,20 @@ emit_declaration(
                              first_inst);
 
    for (idx = first; idx <= last; ++idx) {
-      boolean ok;
-
       switch (decl->Declaration.File) {
       case TGSI_FILE_TEMPORARY:
          for (i = 0; i < NUM_CHANNELS; i++)
             bld->temps[idx][i] = lp_build_alloca(&bld->base);
-         ok = TRUE;
          break;
 
       case TGSI_FILE_OUTPUT:
          for (i = 0; i < NUM_CHANNELS; i++)
             bld->outputs[idx][i] = lp_build_alloca(&bld->base);
-         ok = TRUE;
          break;
 
       default:
          /* don't need to declare other vars */
-         ok = TRUE;
-      }
-
-      if (!ok) {
-         LLVMPositionBuilderAtEnd(bld->base.builder,
-                                  current_block);
-         return FALSE;
+         break;
       }
    }