Skip to content
  • Teresa Johnson's avatar
    Change TargetLibraryInfo analysis passes to always require Function · 9c27b59c
    Teresa Johnson authored
    Summary:
    This is the first change to enable the TLI to be built per-function so
    that -fno-builtin* handling can be migrated to use function attributes.
    See discussion on D61634 for background. This is an enabler for fixing
    handling of these options for LTO, for example.
    
    This change should not affect behavior, as the provided function is not
    yet used to build a specifically per-function TLI, but rather enables
    that migration.
    
    Most of the changes were very mechanical, e.g. passing a Function to the
    legacy analysis pass's getTLI interface, or in Module level cases,
    adding a callback. This is similar to the way the per-function TTI
    analysis works.
    
    There was one place where we were looking for builtins but not in the
    context of a specific function. See FindCXAAtExit in
    lib/Transforms/IPO/GlobalOpt.cpp. I'm somewhat concerned my workaround
    could provide the wrong behavior in some corner cases. Suggestions
    welcome.
    
    Reviewers: chandlerc, hfinkel
    
    Subscribers: arsenm, dschuff, jvesely, nhaehnle, mehdi_amini, javed.absar, sbc100, jgravelle-google, eraman, aheejin, steven_wu, george.burgess.iv, dexonsmith, jfb, asbirlea, gchatelet, llvm-commits
    
    Tags: #llvm
    
    Differential Revision: https://reviews.llvm.org/D66428
    
    llvm-svn: 371284
    9c27b59c
Loading