[PATCH] Bug 14984 - Implement __attribute__((ms_abi))

[Charles Davis]

Sorry about the late reply. Wouldn't want you to think I dropped of the face of the earth, now, would I? :)

On Jun 18, 2013, at 9:23 AM, Reid Kleckner wrote:

> On Tue, Jun 18, 2013 at 11:04 AM, Anton Korobeynikov <anton at korobeynikov.info> wrote:
> > Charles, can you commit http://llvm-reviews.chandlerc.com/D729, or is
> > something blocking that?
I want to commit it, but... well, see below.
> I think we somehow wanted to unify the stuff with ARM which already
> has a way to select different ABIs / calling conventions... But I do
> not know where and how everything stopped.
I replied, "That sounds like a good idea, but I think that should be a different patch." Then I asked what Anton thought. I haven't committed this patch yet because I was waiting for him to reply and say if it was OK or not. He never did, and so we all forgot about it for a while. If you'd like, I will commit that patch right away so we can get on with this.
> 
> Can you elaborate on what ARM does that x86 doesn't?  It seems to have pretty much the same thing.  It has multiple ccs, some subtarget flags, and it checks the cc before checking the subtarget for what to do in the default case.
Like I said, I think that's better saved for a different patch, and another thread, for another day.

Chip

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130624/9ae9ca77/attachment.html>