r180976 - Move parsing of identifiers in MS-style inline assembly into

[Chad Rosier]

On May 21, 2013, at 11:33 AM, Reid Kleckner <rnk at google.com> wrote:

> Chad, is anyone actively working on this, or is this available to work on?  This breaks a bunch of stuff for us.

Thanks for bringing this to my attention, Reid.  I was on vacation last week, so this is the first I'm hearing of it.

Unfortunately, I'm slammed with other work at the moment and I believe John has been sick for the past few days.

I won't have time to investigate this for a few days, so feel free to work on it in the mean time.

 Chad

> 
> 
> On Tue, May 14, 2013 at 6:03 PM, John McCall <rjmccall at apple.com> wrote:
> On May 14, 2013, at 8:25 AM, Hans Wennborg <hans at chromium.org> wrote:
> > On Fri, May 3, 2013 at 1:10 AM, John McCall <rjmccall at apple.com> wrote:
> >> Author: rjmccall
> >> Date: Thu May  2 19:10:13 2013
> >> New Revision: 180976
> >>
> >> URL: http://llvm.org/viewvc/llvm-project?rev=180976&view=rev
> >> Log:
> >> Move parsing of identifiers in MS-style inline assembly into
> >> the actual parser and support arbitrary id-expressions.
> >
> > This seems to have broken the ability to use labels inside MS-style
> > inline assembly. For example, the following no longer compiles:
> >
> >  void foo() {
> >    __asm {
> >  a:
> >      jmp a
> >    }
> >  }
> >
> > because Clang now errors out when it fails to look up the identifier "a".
> >
> > Do you have any ideas for a good way to handle this?
> 
> The interface should probably have a way to say that the frontend just
> didn't recognize a name, and Clang should trigger that when it sees a
> bare identifier that's not in the lookup tables.
> 
> John.
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
> 

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