Re: [Mesa-dev] [PATCH v2 09/25] mesa: move nir_spirv_supported_capabilities definition

2017-12-06 Thread Alejandro Piñeiro
On 06/12/17 10:47, Timothy Arceri wrote: > > > On 06/12/17 20:33, Alejandro Piñeiro wrote: >> On 06/12/17 10:23, Timothy Arceri wrote: >>> Can we get away with forward declaring this? >>> >>> There is a section at the top of mtypes you can add it to: >>> >>>   * \name Some forward type declarations

Re: [Mesa-dev] [PATCH v2 09/25] mesa: move nir_spirv_supported_capabilities definition

2017-12-06 Thread Alejandro Piñeiro
On 06/12/17 10:47, Timothy Arceri wrote: > > > On 06/12/17 20:33, Alejandro Piñeiro wrote: >> On 06/12/17 10:23, Timothy Arceri wrote: >>> Can we get away with forward declaring this? >>> >>> There is a section at the top of mtypes you can add it to: >>> >>>   * \name Some forward type declarations

Re: [Mesa-dev] [PATCH v2 09/25] mesa: move nir_spirv_supported_capabilities definition

2017-12-06 Thread Timothy Arceri
On 06/12/17 20:33, Alejandro Piñeiro wrote: On 06/12/17 10:23, Timothy Arceri wrote: Can we get away with forward declaring this? There is a section at the top of mtypes you can add it to:  * \name Some forward type declarations Yes, I realized that, and tried, but I still got several bui

Re: [Mesa-dev] [PATCH v2 09/25] mesa: move nir_spirv_supported_capabilities definition

2017-12-06 Thread Alejandro Piñeiro
On 06/12/17 10:33, Alejandro Piñeiro wrote: > On 06/12/17 10:23, Timothy Arceri wrote: >> Can we get away with forward declaring this? >> >> There is a section at the top of mtypes you can add it to: >> >>  * \name Some forward type declarations > Yes, I realized that, and tried, but I still got

Re: [Mesa-dev] [PATCH v2 09/25] mesa: move nir_spirv_supported_capabilities definition

2017-12-06 Thread Alejandro Piñeiro
On 06/12/17 10:23, Timothy Arceri wrote: > Can we get away with forward declaring this? > > There is a section at the top of mtypes you can add it to: > >  * \name Some forward type declarations Yes, I realized that, and tried, but I still got several build errors. So that would not be enough. In

Re: [Mesa-dev] [PATCH v2 09/25] mesa: move nir_spirv_supported_capabilities definition

2017-12-06 Thread Timothy Arceri
Can we get away with forward declaring this? There is a section at the top of mtypes you can add it to: * \name Some forward type declarations On 01/12/17 04:28, Eduardo Lima Mitev wrote: From: Alejandro Piñeiro Due gl_spirv we will use it on more places, specifically on gl_constants, wher

[Mesa-dev] [PATCH v2 09/25] mesa: move nir_spirv_supported_capabilities definition

2017-11-30 Thread Eduardo Lima Mitev
From: Alejandro Piñeiro Due gl_spirv we will use it on more places, specifically on gl_constants, where we would like to use it without a pointer. --- src/compiler/spirv/nir_spirv.h | 15 ++- src/mesa/main/mtypes.h | 11 +++ 2 files changed, 13 insertions(+), 13 delet