[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