Re: [PATCH 5/5] d3dcompiler: Store a pointer to the bwriter_shader structure inside the bc_writer structure.

[Matteo Bruni]

On Fri, Mar 20, 2020 at 12:08 AM Zebediah Figura  wrote:
> Signed-off-by: Zebediah Figura
>
> dlls/d3dcompiler_43/bytecodewriter.c      | 30 ++++++++++++++---------
>  dlls/d3dcompiler_43/d3dcompiler_private.h |  3 +--
>  2 files changed, 20 insertions(+), 13 deletions(-)
>
> diff --git a/dlls/d3dcompiler_43/bytecodewriter.c b/dlls/d3dcompiler_43/bytecodewriter.c
> index 8b016cd6201..add9804cac0 100644
> --- a/dlls/d3dcompiler_43/bytecodewriter.c
> +++ b/dlls/d3dcompiler_43/bytecodewriter.c
> @@ -939,7 +939,8 @@ static void vs_12_dstreg(struct bc_writer *This, const struct shader_reg *reg,
>              break;
>      case BWRITERSPR_PREDICATE:
> -
>        if(This->version != BWRITERVS_VERSION(2, 1)){
> -
>        if (This->shader->minor_version == 0)
> -
>        {
>            WARN("Predicate register is allowed only in vs_2_x\n");
>            This->state = E_INVALIDARG;
>            return;

This function is also used for vs_1_* so, technically, this should
check major version too.