[Mesa-dev] [PATCH] radeon/llvm: Use the llvm.rsq.clamped intrinsic for RSQ

Tom Stellard tom at stellard.net
Wed Jul 2 12:09:52 PDT 2014


On Wed, Jul 02, 2014 at 06:29:25PM +0200, Laurent Carlier wrote:
> Le mercredi 25 juin 2014, 11:58:47 Michel Dänzer a écrit :
> > On 25.06.2014 09:15, Tom Stellard wrote:
> > > https://bugs.freedesktop.org/show_bug.cgi?id=80015
> > > 
> > > CC: "10.1 10.2" <mesa-stable at lists.freedesktop.org>
> > > ---
> > > 
> > >  src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c | 2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > > 
> > > diff --git a/src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c
> > > b/src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c index
> > > 217fa32..119e613 100644
> > > --- a/src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c
> > > +++ b/src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c
> > > @@ -1385,7 +1385,7 @@ void radeon_llvm_context_init(struct
> > > radeon_llvm_context * ctx)> 
> > >  	bld_base->rsq_action.emit = build_tgsi_intrinsic_nomem;
> > >  
> > >  #if HAVE_LLVM >= 0x0305
> > > 
> > > -	bld_base->rsq_action.intr_name = "llvm.AMDGPU.rsq.";
> > > +	bld_base->rsq_action.intr_name = "llvm.AMDGPU.rsq.clamped.f32";
> > > 
> > >  #else
> > >  
> > >  	bld_base->rsq_action.intr_name = "llvm.AMDGPU.rsq";
> > >  
> > >  #endif
> > 
> > Reviewed-and-Tested-by: Michel Dänzer <michel.daenzer at amd.com>
> 
> Tested-by: Laurent Carlier <lordheavym at gmail.com>
> 
> Anyone to commit it?

I've committed it.  Thanks for testing.

-Tom

> -- 
> Laurent Carlier
> http://www.archlinux.org



> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev



More information about the mesa-dev mailing list