The LLVM side of this LGTM, with the minor things I mentioned over IRC.

Thanks,
Joey

-----Original Message-----
From: [email protected]
[mailto:[email protected]] On Behalf Of Tim Northover
Sent: 01 October 2013 13:54
To: Aaron Ballman
Cc: llvm-commits; [email protected]
Subject: Re: [PATCH] ARM: Implement __attribute__((interrupt(...)))

> Since this is a GNU attribute, can it be spelled __interrupt__ as
> well, and be expected to work?

I don't believe so. At least, GCC doesn't allow it. I've attached an
updated version of the two patches.

OK to commit?

Cheers.

Tim.




_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to