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

Pete Cooper peter_cooper at apple.com
Mon Jun 22 12:41:38 PDT 2015


> On Jun 22, 2015, at 10:31 AM, Rafael Espíndola <rafael.espindola at gmail.com> wrote:
> 
>> 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.
Thanks, r240318.

I’ve left it as a utility for now which i hope is ok.  If you later have a stronger preference for moving it to MCAsmParser then let me know and i’ll be happy to move it.

Cheers,
Pete
> 
> Cheers,
> Rafael





More information about the llvm-commits mailing list