[PATCH] D12681: Calling conventions for HHVM

[Chad Rosier via llvm-commits]

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