On 20.04.2016 02:52, Tom Stellard wrote: > The range metadata tells LLVM the range of expected values for this intrinsic, > so it can do some additional optimizations on the result. > --- > src/gallium/drivers/radeonsi/si_shader.c | 29 ++++++++++++++++++++++++++--- > 1 file changed, 26 insertions(+), 3 deletions(-) > > diff --git a/src/gallium/drivers/radeonsi/si_shader.c > b/src/gallium/drivers/radeonsi/si_shader.c > index 3b6d6e9..b4f2a42 100644 > --- a/src/gallium/drivers/radeonsi/si_shader.c > +++ b/src/gallium/drivers/radeonsi/si_shader.c > @@ -1114,12 +1114,35 @@ static LLVMValueRef get_sample_id(struct > radeon_llvm_context *radeon_bld) > SI_PARAM_ANCILLARY, 8, 4); > } > > +/** > + * Set range metadata on an instruction. This can only be used on load and > + * call instructions. To specify an instruciton can only produce the values
Typo: instruciton -> instruction It would also be nice to always write LDS instead of lds in the commit log of patch 4, but either way, with the above typo fixed, the series is Reviewed-by: Michel Dänzer <michel.daen...@amd.com> Nice work! -- Earthling Michel Dänzer | http://www.amd.com Libre software enthusiast | Mesa and X developer _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
