[PATCH] D140369: [PowerPC] update PPCTTIImpl::supportsTailCallFor() check conditions

[ChenZheng via Phabricator via llvm-commits]

shchenz accepted this revision as: shchenz.
shchenz added a comment.
This revision is now accepted and ready to land.

LGTM with one nit.



================
Comment at: llvm/lib/Target/PowerPC/PPCISelLowering.cpp:5672
+      CalleeGV, CalleeCC, CallerCC, CB, CalleeFunc->isVarArg(), Outs, Ins,
+      false /*IsByValArg*/, CallerFunc, false /*isCalleeExternalSymbol*/);
+}
----------------
we may need a NFC patch to remove the `IsByValArg` for `isEligibleForTCO` as all callers pass `false` to it, we should be able to check the byval parameter inside `isEligibleForTCO`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140369