[PATCH] D39607: [PartialInliner] Inline vararg functions that forward varargs.

David Li via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 7 09:45:27 PST 2017


davidxl added inline comments.


================
Comment at: lib/Transforms/IPO/PartialInlining.cpp:830
+  // cleanup if we find vastart.
+  if (ClonedFunc->isVarArg() && OutlinedFunc)
+    for (auto &BB : *ClonedFunc)
----------------
Is it possible to teach code extractor to handle this and return null instead of doing the cleanup here?  CodeExtractor knows the set of blocks to be extracted or not.


https://reviews.llvm.org/D39607





More information about the llvm-commits mailing list