On Tuesday, June 20, 2017 10:05:26 AM PDT Ian Romanick wrote: > On 06/20/2017 05:23 AM, Lionel Landwerlin wrote: > > In 5f2fe9302c is_geminilake was introduced for the differenciate > > broxton from geminilake. Unfortunately I failed as verifying that > > is_broxton is throughout the code base to mean Gen9lp. > > It looks like this replaces every instances of devinfo->is_broxton. Why > not just s/devinfo->is_broxton/devinfo->gen9lp/g and change the way the > field is set? This is effectively what the change to brw_context.c does > to brw_context::is_broxton.
I've also wondered about having an is_atom bit, so you could just do if (gen == 9 && is_atom) or something for BYT/BSW/BXT/GLK checks. *shrug*
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
