[llvm-commits] [PATCH] instcombine: wire up -fno-builtin to the library call simplifier
Meador Inge
meadori at codesourcery.com
Thu Nov 8 09:26:43 PST 2012
On Nov 8, 2012, at 1:58 AM, Eli Friedman wrote:
> On Wed, Nov 7, 2012 at 8:24 PM, Meador Inge <meadori at codesourcery.com> wrote:
>> Now that the library call simplifiers are being migrated out of their own
>> pass into the instruction combiner pass the latter needs to be modified to
>> work with -fno-builtin. This is accomplished by adding a new parameter
>> to the InstCombiner constructor and to the createInstructionCombiningPass
>> factory function that controls whether library call simplification should
>> be performed or not. The pass manager already has an option to control
>> library call simplification. This option is now passed through when creating
>> instances of the instruction combiner class.
>>
>> OK?
>
> It would be better if you could just fix the code in question so it
> uses TargetLibraryInfo correctly.
D'oh. I missed where -fno-builtin is already tied to TLI. I will rework things so that TLI
is queried appropriately.
> Also, testcase please.
Will do. Thanks for the feedback.
--
Meador Inge
CodeSourcery / Mentor Embedded
http://www.mentor.com/embedded-software
More information about the llvm-commits
mailing list