[PATCH] D52067: [inline Cost] Don't mark function accessing varargs as non-inlinable
Florian Hahn via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Sep 20 05:12:50 PDT 2018
fhahn accepted this revision.
fhahn added a comment.
This revision is now accepted and ready to land.
LGTM, with my previous comments.
================
Comment at: lib/Analysis/InlineCost.cpp:1242
case Intrinsic::vastart:
- case Intrinsic::vaend:
UsesVarArgs = true;
return false;
----------------
Maybe it would make sense to change this to `InitsVarArgs` or something now
https://reviews.llvm.org/D52067
More information about the llvm-commits
mailing list