[Mesa-dev] Mesa (master): gallium: Add tgsi_to_nir to get a nir_shader for a TGSI shader.

[Connor Abbott]

On Wed, Apr 1, 2015 at 2:53 PM, Brian Paul <brianp at vmware.com> wrote:
> On 04/01/2015 12:49 PM, Brian Paul wrote:
>>
>> On 04/01/2015 12:02 PM, Eric Anholt wrote:
>>>
>>> Module: Mesa
>>> Branch: master
>>> Commit: 783ad697d25e754ab719ab6c715969c35dbe867b
>>> URL:
>>>
>>> https://urldefense.proofpoint.com/v2/url?u=http-3A__cgit.freedesktop.org_mesa_mesa_commit_-3Fid-3D783ad697d25e754ab719ab6c715969c35dbe867b&d=AwIGaQ&c=Sqcl0Ez6M0X8aeM67LKIiDJAXVeAw-YihVMNtXt-uEs&r=T0t4QG7chq2ZwJo6wilkFznRSFy-8uDKartPGbomVj8&m=a-09zLFxtZs6ixn-alr13iDBorBkDSkqkMg6vkd_pck&s=saYPnwqarUJ3EYKew21KmGPcWAUJEnLemc79_M8o9Rs&e=
>>>
>>>
>>> Author: Eric Anholt <eric at anholt.net>
>>> Date:   Wed Oct 29 14:32:16 2014 -0700
>>>
>>> gallium: Add tgsi_to_nir to get a nir_shader for a TGSI shader.
>>>
>>> This will be used by the VC4 driver for doing device-independent
>>> optimization, and hopefully eventually replacing its whole IR.  It also
>>> may be useful to other drivers for the same reason.
>>>
>>> v2: Add all of the instructions I was relying on tgsi_lowering to remove,
>>>      and more.
>>> v3: Rebase on SSA rework of the builder.
>>> v4: Use the NIR ineg operation instead of doing a src modifier.
>>> v5: Don't use ineg for fnegs.  (infer_src_type on MOV doesn't do what I
>>>      expect, again).
>>> v6: Fix handling of multi-channel KILL_IF sources.
>>> v7: Make ttn_get_f() return a swizzle of a scalar load_const, rather than
>>>      a vector load_const.  CSE doesn't recognize that srcs out of those
>>>      channels are actually all the same.
>>> v8: Rebase on nir_builder auto-sizing, make the scalar arguments to
>>>      non-ALU instructions actually be scalars.
>>> v9: Add support for if/loop instructions, additional texture targets, and
>>>      untested support for indirect addressing on temps.
>>> v10: Rebase on master, drop bad comment about control flow and just
>>> choose
>>>       the X channel, use int comparison opcodes in LIT for now, drop
>>> unused
>>>       pipe_context argument..
>>> v11: Fix translation of LRP (previously missed because I mis-translated
>>>       back out), use nir_builder init helpers.
>>> v12: Rebase on master, adding explicit include of mtypes.h to get
>>>       INTERP_QUALIFIER_*
>>> v13: Rebase on variables being in lists instead of hash tables, drop use
>>>       of mtypes.h in favor of util/pipeline.h.  Use Ken's nir_builder
>>>       swizzle and fmov/imov_alu helpers, drop "struct" in front of
>>>       nir_builder, use nir_builder directly as the function arg in a
>>> lot of
>>>       cases, drop redundant members of ttn_compile that are also in
>>>       nir_builder, drop some half-baked malloc failure handling.
>>> v14: The indirect uniform src0 should be scalar, not vector (noticed as
>>>       odd by robclark, confirmed by cwabbott).  Apply Ken's review to
>>>       initialize s->num_uniforms and friends, skip ttn_channel for dot
>>>       products, and use the simpler discard_if intrinsic.
>>>
>>> Reviewed-by: Kenneth Graunke <kenneth at whitecape.org> (v13)
>>> Acked-by: Rob Clark <robclark at freedesktop.org>
>>
>>
>> For some reason, the scons build is not working after this commit on
>> Windows and some of our Linux machines.  But it does build on my
>> personal Linux machine:
>>
>> The error is:
>>
>> scons: done reading SConscript files.
>> scons: Building targets ...
>>    Compiling src/gallium/auxiliary/nir/tgsi_to_nir.c ...
>> In file included from src/gallium/auxiliary/nir/tgsi_to_nir.c:26:0:
>> src/glsl/nir/nir.h:39:25: fatal error: nir_opcodes.h: No such file or
>> directory
>>   #include "nir_opcodes.h"
>>                           ^
>> compilation terminated.
>>
>> I've added '#/src/glsl/nir' to the CPPPATH in
>> src/gallium/auxiliary/SConscript but it's still not working.
>>
>> I'll keep digging.
>
>
> OK, nir_opcodes.h is a generated file and I had a copy left over from a
> previous make.  We need to add SConscript code to generate the file. I'll
> take a stab at it but would be happy if someone beat me to it.

You probably already know this, but note that there are a few more
Python-generated files:

nir_opcodes.c
nir_constant_expressions.c
nir_opt_algebraic.c
nir_builder_opcodes.h

which you can see from looking at src/glsl/Makefile.am.

>
>
> -Brian
>
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev