On Thu, Nov 5, 2015 at 8:18 AM, Emil Velikov <emil.l.veli...@gmail.com> wrote: > From: Emil Velikov <emil.veli...@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. _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
