[llvm-commits] [llvm] r140852 - in /llvm/trunk: lib/Transforms/InstCombine/InstCombine.h lib/Transforms/InstCombine/InstructionCombining.cpp test/Transforms/InstCombine/LandingPadClauses.ll

[Bill Wendling]

On Sep 30, 2011, at 11:19 AM, Chris Lattner wrote:

> On Sep 30, 2011, at 6:12 AM, Duncan Sands wrote:
> 
>> Author: baldrick
>> Date: Fri Sep 30 08:12:16 2011
>> New Revision: 140852
>> 
>> URL: http://llvm.org/viewvc/llvm-project?rev=140852&view=rev
>> Log:
>> Inlining often produces landingpad instructions with repeated
>> catch or repeated filter clauses.  Teach instcombine a bunch
>> of tricks for simplifying landingpad clauses.  Currently the
>> code only recognizes the GNU C++ and Ada personality functions,
>> but that doesn't stop it doing a bunch of "generic" transforms
>> which are hopefully fine for any real-world personality function.
>> If these "generic" transforms turn out not to be generic, they
>> can always be conditioned on the personality function.  Probably
>> someone should add the ObjC++ personality function.  I didn't as
>> I don't know anything about it.
> 
> Very cool!  Can you split the bodies of the large for loops out into static helper functions?
> 
> Bill, can you audit this and enhance it to support the ObjC++ personality function?
> 
Sure.

-bw