[PATCH][ARM] Fix .thumb_func directive in ARMAsmParser

Renato Golin renato.golin at linaro.org
Tue Oct 22 13:26:44 PDT 2013


On 22 October 2013 18:32, Gabor Ballabas <gaborb at inf.u-szeged.hu> wrote:

> I didn't add any new test just modified the existing one by moving the
> .type directive between the .thumb_func directive
> and the actual symbol (you can check it in the diff). This is how GCC
> places the directives (at least it did it for me when
> I compiled  a simple "Hello world!" C program to assembly code).
>

Hi Gabor,

This seems to make sense, but the patch is a bit hard to crack, given no
context and many small functions around.

Can you create the patch with -U30 and post on Phabricator?

thanks,
--renato
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131022/46047693/attachment.html>


More information about the llvm-commits mailing list