[PATCH] AsmParser: add support for .end directive
Renato Golin
renato.golin at linaro.org
Mon Dec 16 12:07:22 PST 2013
On 16 December 2013 17:13, Saleem Abdulrasool <compnerd at compnerd.org> wrote:
> That was in the commit message: the glaring problem with this
> implantation is that the flush is a huge sore thumb. I wasn't sure if it
> is worth adding a discard buffer call. I figured that adding that later is
> definitely possible so I would keep the patch simple.
>
Ok, I seem to have misread that, too... It was too early in the morning,
sorry.
I think it depends if there are any side effects to calling Lex() that
wouldn't be done with a Discard() (like counting number of tokens or
something like that). I don't really know, Clang folks can make sure you're
not invalidating any premise, but I'd vote in favour of a new discard
method.
I was being goofy. I can change it if you want.
>
No, that's ok, I was just unaware of the tarp meme. ;)
cheers,
--renato
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131216/e057fb63/attachment.html>
More information about the llvm-commits
mailing list