search for: numannotated

...nterface as well? That makes sense to me. Yup, that's it. >> An option for enabling/disabling library call simplification in `InstCombiner` >> will be available. For backwards compatibility perhaps it should remain >> '-simplify-libcalls'. The `NumSimplified` and `NumAnnotated` statistics shall >> be added to `InstCombiner`. > > There is no need to preserver the -simplify-libcalls flag. It's an internal > implementation detail of the compiler, one that is better left in the past :) Do you think we still need some way to disable just the library call...

...ador Inge <meadori at codesourcery.com> wrote: >>> An option for enabling/disabling library call simplification in `InstCombiner` >>> will be available. For backwards compatibility perhaps it should remain >>> '-simplify-libcalls'. The `NumSimplified` and `NumAnnotated` statistics shall >>> be added to `InstCombiner`. >> >> There is no need to preserver the -simplify-libcalls flag. It's an internal >> implementation detail of the compiler, one that is better left in the past :) > > Do you think we still need some way to dis...

.... This will done incrementally one optimization at a time to make things easier to review. An option for enabling/disabling library call simplification in `InstCombiner` will be available. For backwards compatibility perhaps it should remain '-simplify-libcalls'. The `NumSimplified` and `NumAnnotated` statistics shall be added to `InstCombiner`. As a result of this migration 'Transforms/Scalar/SimplifyLibCalls.cpp' will go away. Proof of Concept ================ A very rough proof of concept patch has been attached to this proposal showing how the table-driven method can be integrate...

...an be moved in larger chunks (e.g. by family) instead of each individually. > An option for enabling/disabling library call simplification in `InstCombiner` > will be available. For backwards compatibility perhaps it should remain > '-simplify-libcalls'. The `NumSimplified` and `NumAnnotated` statistics shall > be added to `InstCombiner`. There is no need to preserver the -simplify-libcalls flag. It's an internal implementation detail of the compiler, one that is better left in the past :) > As a result of this migration 'Transforms/Scalar/SimplifyLibCalls.cpp' wil...

----- Original Message ----- > From: "James Molloy via llvm-dev" <llvm-dev at lists.llvm.org> > To: "Vaivaswatha Nagaraj" <vn at compilertree.com> > Cc: "LLVM Dev" <llvm-dev at lists.llvm.org> > Sent: Thursday, December 3, 2015 4:41:46 AM > Subject: Re: [llvm-dev] Function attributes for LibFunc and its impact on GlobalsAA > >