[PATCH] D21282: AMDGPU: mark {exp, log}10{, f, l} library functions as unavailable
Nicolai Hähnle via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 14 06:19:42 PDT 2016
nhaehnle added a comment.
I think it's mostly used by SimplifyLibCalls and perhaps other parts of InstCombine to decide whether certain transformations are allowed.
Interestingly, NVPTX sets disableAllFunctions, but there is a comment saying that "having no standard library prevents e.g. many fastmath optimizations". And indeed, on my shader-db using disableAllFunctions() makes a difference. It might be limited to sqrt and abs functions.
I'll commit this change as-is for now, i.e. stick with blacklisting.
http://reviews.llvm.org/D21282
More information about the llvm-commits
mailing list