[PATCH] D12681: Calling conventions for HHVM

Chad Rosier via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 14 15:42:30 PDT 2015


mcrosier added inline comments.

================
Comment at: lib/Target/X86/X86ISelLowering.cpp:3874
@@ +3873,3 @@
+
+  if (IsTailCallConvention(CallingConv))
+    return IsVarArg ? false : TailCallOpt;
----------------
maksfb wrote:
> mcrosier wrote:
> > While this looks correct, I'm not sure I agree with the refactoring.
> Which part of it? The logic here is to check for a tail call calling convention, and I thought it'd better to maintain a single list of those.
Nevermind. You make a good point. :)


http://reviews.llvm.org/D12681





More information about the llvm-commits mailing list