On Wed, Sep 30, 2015 at 12:58 AM, Kenneth Graunke <kenn...@whitecape.org> wrote: > Previously, we used nir_lower_io with the scalar type_size function, > which mapped VERT_ATTRIB_* locations to...some numbers. Then, in > fs_visitor::nir_setup_inputs(), we created temporaries indexed by > those numbers, and emitted MOVs from the actual ATTR registers to > those temporaries. > > This patch reworks our input lowering to produce NIR lower_input > intrinsics that properly index into the ATTR file, so we can access > it directly. > > No changes in shader-db.
You might add a sentence here that explains that this patch lets the driver avoid generating a bunch of MOVs that would otherwise be optimized away. The contrasting "avoid tons of MOVs" and "No changes in shader-db" might be confusing otherwise. _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev