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

Nicolai Hähnle via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 19 09:08:15 PST 2015


nhaehnle added a comment.

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.


http://reviews.llvm.org/D14327





More information about the llvm-commits mailing list