On 06.07.2015 20:41, Marek Olšák wrote: > I thought your patch does something else, but after re-reading it, > it seems to do the right thing.
Yeah, si_llvm_export_vs's param_count only counts parameter exports. > Reviewed-by: Marek Olšák <marek.ol...@amd.com> Thanks. > On Mon, Jul 6, 2015 at 1:38 PM, Marek Olšák <mar...@gmail.com> wrote: >> >> CLIPDIST is special in that it's exported twice, first as POS and >> then as PARAM, the same will be done for CULLDIST in the future, so >> these two should indeed be removed from the list. With this change, you won't need to do anything in si_shader_vs when making that change for CULLDIST, si_llvm_export_vs just needs to include it in param_count. -- Earthling Michel Dänzer | http://www.amd.com Libre software enthusiast | Mesa and X developer _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
