[PATCH] D14327: Add llvm.ldexp.* intrinsic, associated SDNode and library calls

[Tom Stellard via llvm-commits]

tstellarAMD added a comment.

In http://reviews.llvm.org/D14327#292935, @nhaehnle wrote:

> In http://reviews.llvm.org/D14327#292904, @tstellarAMD wrote:
>
> > Couldn't the original bug be fixed by marking ldexpf as unavailable for AMDGPU in lib/Analysis/TargetLibraryInfo.cpp ?
>
>
> I think so, yes. Though Matt said that we do want to use the ldexp instruction because it is a full-rate instruction.


Ok, so for a temporary solution, rather than changing the intrinsic emitted by Mesa, I think we should mark this libcall as unavailable.  This current patch could then be done as a follow up.


http://reviews.llvm.org/D14327