[PATCH] D54270: [ThinLTO] Update handling of vararg functions to match inliner

Teresa Johnson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 8 10:34:10 PST 2018


tejohnson created this revision.
tejohnson added a reviewer: davidxl.
Herald added subscribers: dexonsmith, steven_wu, eraman, inglorion, mehdi_amini.

Previously we marked all vararg functions as non-inlinable in the
function summary, which prevented their importing. However, the
corresponding inliner restriction was loosened in r321940/r342675
to only apply to functions calling va_start. Adjust the summary
flag computation to match.


Repository:
  rL LLVM

https://reviews.llvm.org/D54270

Files:
  lib/Analysis/ModuleSummaryAnalysis.cpp
  test/Bitcode/thinlto-function-summary.ll
  test/Transforms/FunctionImport/Inputs/funcimport.ll
  test/Transforms/FunctionImport/funcimport.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D54270.173193.patch
Type: text/x-patch
Size: 5965 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181108/d960815e/attachment.bin>


More information about the llvm-commits mailing list