[PATCH] D109104: [CSSPGO] Allow inlining recursive call for preinliner

Hongtao Yu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 2 09:48:10 PDT 2021


hoy accepted this revision.
hoy added a comment.
This revision is now accepted and ready to land.

lgtm



================
Comment at: llvm/lib/Analysis/InlineCost.cpp:939
+        Writer(this) {
+    AllowRecursiveCall = Params.AllowRecursiveCall.getValue();
+  }
----------------
Nit: move this to the field initializer list? `AllowRecursiveCall(Params.AllowRecursiveCall.getValue())`


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D109104/new/

https://reviews.llvm.org/D109104



More information about the llvm-commits mailing list