Re: [Mesa-dev] [PATCH] nir/spirv: handle if's with same label in both branches

2017-09-11 Thread Samuel Iglesias Gonsálvez
On 09/07/2017 07:03 PM, Jason Ekstrand wrote: > On Thu, Aug 24, 2017 at 8:16 AM, Juan A. Suarez Romero > > wrote: > > When a conditional branch has the same labels in the "if" part and > in the > "else" part, then we have the same cfg

Re: [Mesa-dev] [PATCH] nir/spirv: handle if's with same label in both branches

2017-09-07 Thread Jason Ekstrand
On Thu, Aug 24, 2017 at 8:16 AM, Juan A. Suarez Romero wrote: > When a conditional branch has the same labels in the "if" part and in the > "else" part, then we have the same cfg block, and it must be handled > once. > > Fixes: >

Re: [Mesa-dev] [PATCH] nir/spirv: handle if's with same label in both branches

2017-09-07 Thread Juan A. Suarez Romero
This patch is still unreviewed. Gently ping if someone can take a look. Thanks in advance! J.A. On Thu, 2017-08-24 at 17:16 +0200, Juan A. Suarez Romero wrote: > When a conditional branch has the same labels in the "if" part and in the > "else" part, then we have the same cfg block,

[Mesa-dev] [PATCH] nir/spirv: handle if's with same label in both branches

2017-08-24 Thread Juan A. Suarez Romero
When a conditional branch has the same labels in the "if" part and in the "else" part, then we have the same cfg block, and it must be handled once. Fixes: dEQP-VK.spirv_assembly.instruction.compute.conditional_branch.same_labels*