[PATCH 4/5] d3dcompiler: Always use vkd3d-shader's preprocessor.

[Matteo Bruni]

On Mon, Mar 28, 2022 at 4:59 PM Zebediah Figura <zfigura at codeweavers.com> wrote:
>
> On 3/28/22 07:12, Matteo Bruni wrote:
> > diff --git a/dlls/d3dx9_36/tests/asm.c b/dlls/d3dx9_36/tests/asm.c
> > index 9b4e82f218a..bfc84b2f9c2 100644
> > --- a/dlls/d3dx9_36/tests/asm.c
> > +++ b/dlls/d3dx9_36/tests/asm.c
> > @@ -259,7 +259,7 @@ static void assembleshader_test(void)
> >       messages = NULL;
> >       hr = D3DXAssembleShader(testshader2, strlen(testshader2), NULL, &include.ID3DXInclude_iface,
> >                               D3DXSHADER_SKIPVALIDATION, &shader, &messages);
> > -    ok(hr == D3D_OK, "D3DXAssembleShader test failed with error 0x%x - %d\n", hr, hr & 0x0000FFFF);
> > +    todo_wine ok(hr == D3D_OK, "D3DXAssembleShader test failed with error 0x%x - %d\n", hr, hr & 0x0000FFFF);
> >       if(messages) {
> >           trace("recursive D3DXAssembleShader messages:\n%s", (char *)ID3DXBuffer_GetBufferPointer(messages));
> >           ID3DXBuffer_Release(messages);
>
> What's the cause of this failure?

There is no define for "REGISTER" there so it ends up unchanged in the
preprocessed source. Somehow native doesn't flinch and returns S_OK
anyway (IIRC the output shader has no actual instructions, just the
version token and the end token)...