Where's the patchlist on your cover letter? :)
1: Reviewed-by: Jordan Justen <jordan.l.jus...@intel.com>
2: If only changing the string calls:
Reviewed-by: Jordan Justen <jordan.l.jus...@intel.com>
3-5: Reviewed-by: Jordan Justen <jordan.l.jus...@intel.com>
6-8: Do you have data that removing the inlines doesn't hurt things?
;) I actually don't have an opinion on these, so I'll hold off
to see if anyone else chimes in.
9: Reviewed-by: Jordan Justen <jordan.l.jus...@intel.com>
(Noting that you are changing an inline function. ;)
10-13: Reviewed-by: Jordan Justen <jordan.l.jus...@intel.com>
On 2017-08-10 15:02:31, Matt Turner wrote:
> Just a simple series cleaning up various bits of code that clang warns about.
>
> The only real change is to "i965/vec4: Return float from
> spill_cost_for_type()" which fixes an obvious mistake. The rest just cleans
> things up like marking internal functions static.
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
