[PATCH] Optimize call to sqrt.

Nick Lewycky nicholas at mxc.ca
Thu Jun 20 13:00:36 PDT 2013


Chris Lattner wrote:
>
> On Jun 19, 2013, at 8:12 AM, Meador Inge<meadori at codesourcery.com>  wrote:
>
>> On 06/18/2013 03:37 PM, Eli Friedman wrote:
>>
>>> Scalar/SimplifyLibCalls.cpp is basically already dead; I'm surprised nobody
>>> deleted it already.
>>
>> I have a patch in my local queue that removes it, but as mentioned here I am
>> not quite sure what to do about the C API [1].  My current patch just makes
>> LLVMPassManagerBuilderSetDisableSimplifyLibCalls a NOP.
>
> This approach makes sense to me.

How about making it call TargetLibraryInfo::disableAllFunctions()? That 
may take some setup if the TLI doesn't exist yet.

Or how about just removing it entirely? I'd prefer removing it.

Nick



More information about the llvm-commits mailing list