[PATCH] MC: do not try to parse invalid handled directives
Matheus Almeida
matheus.almeida at imgtec.com
Mon Dec 16 03:01:34 PST 2013
Hum... I'm not sure we need this patch.
I think what is happening here is that some target assembly parsers are not being coherent w.r.t return values from ParseDirective. My understanding is that:
True should be returned if a target parser 'doesn't care' about a directive and it should be handled by the generic parser;
False should be returned if a directive was handled by a target parser independently if successfully parsed. Note that the Error routine sets "HadError" as a side effect which means the assembly parser will return an error value if there was a problem parsing a directive.
I think the problem is that some target assembly parsers are returning "True" and handling target specific directives which isn't entirely correct.
Regards,
Matheus
http://llvm-reviews.chandlerc.com/D2410
More information about the llvm-commits
mailing list