[Mesa-dev] GLSL compiler function call cleaning

Eric Anholt eric at anholt.net
Thu Mar 29 10:15:24 PDT 2012


On Wed, 28 Mar 2012 20:32:59 -0700, Kenneth Graunke <kenneth at whitecape.org> wrote:
> I finally resurrected this work I started back in September.  I think
> I've included most of the old feedback, and done a bunch more cleaning.

Patches 2,4,5,6,8,9 are also:

Reviewed-by: Eric Anholt <eric at anholt.net>

> The one outstanding concern I have is IR->Mesa and IR->TGSI: both seem
> to support function calls, but I'm not sure how that all works.  IMHO
> Mesa IR really shouldn't support them---we should just rely on the GLSL
> compiler's inliner.  I'm incredibly skeptical that the code in IR->Mesa
> actually works, since it has a comment that functions other than main()
> should never been seen.  Maybe that's just old.  Can we delete this?

There may exist code in ir_to_mesa, but I don't think that code has been
run, nor do I expect the Mesa optimization passes to do anything good to
programs using the CAL opcode.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20120329/8f47009b/attachment.pgp>


More information about the mesa-dev mailing list