<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div><br><blockquote type="cite"><div dir="ltr">
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Aug 28, 2013 at 9:40 PM, Charles Davis <span dir="ltr"><<a href="mailto:cdavis5x@gmail.com" target="_blank">cdavis5x@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><br><div><div><div class="h5"><div>On Aug 19, 2013, at 3:15 PM, Charles Davis wrote:</div>
<br><blockquote type="cite"><div style="word-wrap:break-word"><br><div><div>On Aug 19, 2013, at 2:06 PM, Reid Kleckner wrote:</div><br><blockquote type="cite"><div dir="ltr">On Thu, Jul 25, 2013 at 7:27 PM, Charles Davis <span dir="ltr"><<a href="mailto:cdavis5x@gmail.com" target="_blank">cdavis5x@gmail.com</a>></span> wrote:<br>
<div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">Actually, it's whatever the default calling convention is. Normally, that's __cdecl, unless you pass -mrtd; then it's __stdcall. It's still ignored on i386; it's just that gcc won't warn on it anymore (for some reason). I haven't been able to find anything in their Bugzilla about it. (This is at least true on 4.7, but I really don't feel like building 4.8 to find out if that's changed. It took all day just for the computer running Linux to build 4.7 so I could verify this for you. ;)<br>
<div><br></div><div>The question is, since it has no effect on any platform except x86-64, should *we* warn about it? I think we should.</div><div><br></div><div>I have a new patch, but I need to test it first. I'm also tired. I'll have it for you tomorrow morning (hopefully).</div>
</div></blockquote><div><br></div><div> Any news on this front?</div></div></div></div>
</blockquote></div>Yeah, but it's bad news.<div><br></div><div>I haven't yet been able to figure out how to make Clang think cdecl is compatible with whatever calling convention (ms_abi or sysv_abi) is the default without breaking a bunch of other stuff. I was waiting for your CC_Default patch to go in, because I was hoping that would make it easier.</div>
<div><br></div><div>Maybe it would just be easier for now to do what GCC does, and just declare that cdecl is compatible with both no matter which target we're targeting.</div><div><br></div><div>Since I've had no luck working on this, I've had this set aside for now. I'll attach what I've got for now, but (since you're still busy with the CC_Default patch) I don't imagine you'll have a lot of time to work on this. I'll see if I can make this work in the meantime.</div>
</div></blockquote></div></div>Now that your CC_Default patch is in, I've had better luck.</div><div><br></div><div>This patch passes all the tests I added. PTAL.</div><div><br></div><div>Chip</div><div></div></div><br>
<div style="word-wrap:break-word"><div></div></div><br></blockquote></div><br></div>
</blockquote></div><br></body></html>