[LLVMbugs] [Bug 2973] New: very poor choice by the inliner (code size + perf)

[bugzilla-daemon at cs.uiuc.edu]

http://llvm.org/bugs/show_bug.cgi?id=2973

           Summary: very poor choice by the inliner (code size + perf)
           Product: libraries
           Version: 1.0
          Platform: PC
        OS/Version: All
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Interprocedural Optimizations
        AssignedTo: unassignedbugs at nondot.org
        ReportedBy: clattner at apple.com
                CC: llvmbugs at cs.uiuc.edu


SPASS has code that looks like this:

LIST clause_NumberSort(LIST List) {
  return list_Sort(List, (BOOL (*) (POINTER, POINTER)) clause_NumberLower);
}

LIST pcheck_ClauseNumberMergeSort(LIST L) {
  return clause_NumberSort(L);
}

Because the inliner is running bottom up, it inlines clause_NumberSort into
pcheck_ClauseNumberMergeSort, duplicating it!  It would be much better to not
inline clause_NumberSort in this case, and just inline
pcheck_ClauseNumberMergeSort into its callers instead.


-- 
Configure bugmail: http://llvm.org/bugs/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.