<div dir="ltr">phab seems down, but here's what I tried to add there:<div><br></div><div>Thanks for the heads-up. WebAssembly's current varargs calling convention does pass the fixed args in one way (via declared function arguments) and the non-fixed args in another way (in memory), so in the cases where NumFixedArgs was wrong we would have been doing the wrong thing. However wasm also typechecks all calls against the declared function signature, so any wrong cases would fail in obvious ways (a signature mismatch error at validation time for direct calls, and at runtime for indirect calls). So just making the behavior correct should be the right thing to do. Also we have explicitly promised that we don't have a stable ABI yet, so that helps too :D<br></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr">On Fri, Sep 22, 2017 at 11:23 AM Eli Friedman via Phabricator via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">efriedma added subscribers: sdardis, sunfish, jpienaar, efriedma.<br>
efriedma added a comment.<br>
<br>
x86, arm, and aarch64 have calling conventions which care whether a call is varargs, but not whether a particular argument is specified in the prototype.<br>
<br>
Some grepping shows that MIPS, WebAssembly and Lanai care about NumFixedArgs.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D37898" rel="noreferrer" target="_blank">https://reviews.llvm.org/D37898</a><br>
<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div>