[PATCH] Change .thumb_set to have the same error checks as .set

Rafael Espíndola rafael.espindola at gmail.com
Mon Jun 22 10:31:24 PDT 2015


> Oh wait, i was wrong.
>
> So AsmParser.cpp doesn’t have a header file.  Its a subclass of MCAsmParser.
> ARMAsmParser itself is a parser extension, but (obviously given there’s no
> header), there’s no way to get from a parser extension to AsmParser.
>
> Everything is connected once you’re down at the MC level, but I didn’t want
> to expose this functionality in MCAsmParser as it doesn’t fit in with the
> other methods there.

I guess I would have a small preference for making
parseAssignmentExpression a method of MCAsmParser, but the current
patch LGTM.

Cheers,
Rafael




More information about the llvm-commits mailing list