On 15 July 2017 at 02:39, Ian Romanick <i...@freedesktop.org> wrote:
> From: Ian Romanick <ian.d.roman...@intel.com>
>
> The old table based spirv_*_to_string functions would return NULL for
> any values "inside" the table that didn't have entries.  The tables also
> needed to be updated by hand each time a new spirv.h was imported.
> Generate the file instead.
>
> Signed-off-by: Ian Romanick <ian.d.roman...@intel.com>
> Suggested-by: Jason Ekstrand <ja...@jlekstrand.net>
> ---
> I am pretty sure the scons changes are bogus... halp, plz?
>
From a quick look the following should do it. Can resend as inline,
etc - let me know what you prefer.

https://github.com/evelikov/Mesa/commit/d8d808db279e8edba441c3416c87324a87a3bff5.patch

>  src/compiler/Makefile.am           |   2 +
>  src/compiler/Makefile.nir.am       |   1 +
>  src/compiler/Makefile.sources      |   4 +-
>  src/compiler/Makefile.spirv.am     |  31 ++++++++
>  src/compiler/SConscript            |   1 +
>  src/compiler/SConscript.spirv      |  32 ++++++++
>  src/compiler/spirv/spirv_info.c    | 156 
> -------------------------------------
>  src/compiler/spirv/spirv_info_c.py |  90 +++++++++++++++++++++
>  8 files changed, 160 insertions(+), 157 deletions(-)
>  create mode 100644 src/compiler/Makefile.spirv.am
>  create mode 100644 src/compiler/SConscript.spirv
>  delete mode 100644 src/compiler/spirv/spirv_info.c
>  create mode 100644 src/compiler/spirv/spirv_info_c.py
>
I think you missed "git add spirv....json"

About keeping the JSON + header file in sync one could do try the following:
 - Add simple bash/other script which parses the revision (other
unique field) from the files and error out on mismatch.
 - Wire said script to `make check'
 - ...
 - Profit :-)

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

Reply via email to