On 18.04.2017 02:09, Timothy Arceri wrote:
V2: some bug fixes and suggestions addressed as suggested by Brian and Nicolai.This series adds some initial support for the KHR_no_error. For now it can only be enabled with the MESA_NO_ERROR environment variable. To start with I've added support to some of the api calls that were taking a long time (or where caled often) in the Civ6 benchmark.
Just as a matter of general musing, I wonder if in some cases the no_error version of an API entry point can be directly configured in the dispatch table. I.e., have a _mesa_no_error_Foo function which the _mesa_Foo function chains to. Though often you have lookups in both cases. Perhaps this is a rare case where templates are actually useful? Having a "no_error" template argument so that the compiler creates two function versions for us automatically, if such a thing were possible in C. Something to consider for later perhaps.
I think patch #1 needs a security check. I also sent a minor comment on patch #4, apart from that patches 2-9, 11, 12 are
Reviewed-by: Nicolai Hähnle <[email protected]>
_______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
-- Lerne, wie die Welt wirklich ist, Aber vergiss niemals, wie sie sein sollte. _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
