[Mesa-dev] [PATCH 13/13] i965/fs_visitor: do not explicitly set outputs[x].file

Matt Turner mattst88 at gmail.com
Thu Nov 5 11:15:39 PST 2015


On Thu, Nov 5, 2015 at 8:18 AM, Emil Velikov <emil.l.velikov at gmail.com> wrote:
> From: Emil Velikov <emil.velikov at collabora.co.uk>
>
> This commits partially reverts "i965: Initialize registers' file to
> BAD_FILE."
>
> No longer needed as of commit "i965/fs: properly construct fs_reg" which
> removes the memset(...0...), and correctly sets BAD_FILE.

I don't think the problem is that the constructors are doing the wrong
thing by memsetting the object. The problem is that the constructor
*isn't being called* :)

fs_reg::init() indeed memsets, but it also sets file = BAD_FILE
(today, and after my series).

So I think the code you're removing is still needed because the
problem of not calling the constructors remain.

Maybe I've misunderstood something.


More information about the mesa-dev mailing list