[PATCH] D14024: [X86] Clean up the tail call eligibility logic

Hans Wennborg via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 23 12:34:31 PDT 2015


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

LGTM, thanks!


================
Comment at: lib/Target/X86/X86ISelLowering.cpp:2475
@@ +2474,3 @@
+static bool mayTailCallThisCC(CallingConv::ID CC) {
+  return isCCallConvention(CC) || canGuaranteeTCO(CC);
+}
----------------
ultra nit: I'd put these in the other order, so the guaranteed ones comes first, and the others read as an addition to those.


http://reviews.llvm.org/D14024





More information about the llvm-commits mailing list