On Tuesday, October 18, 2016 4:38:17 PM PDT Brian Paul wrote:
> Hi Ken,
> 
> I found that this patch causes a regression.  There's a Windows medical 
> app which fails to link some shaders since this change.
> 
> Basically, when the gl_Position VS input is declared as invariant the 
> linker fails with:
> 
> error: declarations for uniform `gl_ModelViewProjectionMatrix' have 
> mismatching invariant qualifiers
> 
> I haven't investigated how to fix this.  I'm hoping you can see a simple 
> fix.
> 
> The attached piglit shader_runner script demonstrates the issue.  Passes 
> w/ NVIDIA.
> 
> Thanks!
> 
> -Brian

Oh, sorry about that!  Here are two possible fixes:

https://cgit.freedesktop.org/~kwg/mesa/commit/?h=invariant-fix
https://cgit.freedesktop.org/~kwg/mesa/commit/?h=invariant-fix-2

They're both kind of hacks...but the whole invariant propagation pass
is kind of a hack, and we've got some other hacks in place already.
So...maybe best to pile another one on.  Not sure which though.

Maybe Jason or Curro will have an opinion...

--Ken

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to