The assert checks whether pshader->num_arrays != 0, but the code
after the assert actually branches based on the same check.

Removing this assert fixes:
  piglit spec@arb_gpu_shader5@execution@samplemaskin-indirect
---
 src/gallium/drivers/r600/sb/sb_bc_parser.cpp | 2 --
 1 file changed, 2 deletions(-)

diff --git a/src/gallium/drivers/r600/sb/sb_bc_parser.cpp 
b/src/gallium/drivers/r600/sb/sb_bc_parser.cpp
index ae92a767b4..52ee3f620b 100644
--- a/src/gallium/drivers/r600/sb/sb_bc_parser.cpp
+++ b/src/gallium/drivers/r600/sb/sb_bc_parser.cpp
@@ -127,8 +127,6 @@ int bc_parser::parse_decls() {
 
        if (pshader->indirect_files & ~((1 << TGSI_FILE_CONSTANT) | (1 << 
TGSI_FILE_SAMPLER))) {
 
-               assert(pshader->num_arrays);
-
                if (pshader->num_arrays) {
                        for (unsigned i = 0; i < pshader->num_arrays; ++i) {
                                r600_shader_array &a = pshader->arrays[i];
-- 
2.13.5

_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to