[LLVMdev] ambiguity of .align

[Rafael Espindola]

2010/1/25 Sandeep Patel <deeppatel1987 at gmail.com>:
> Assuming you're working with an ARM target, you may also hit a problem
> with the alignment option on the .comm directive.
>
> Attached is a first-cut patch for this latter problem.

I have updated your patch to reflect the recent changes. I have also
updated the tests. I have checked that it produces
---------------------------
.comm   bar,75,128
----------------------------

and

---------------------
        .align  7
foo:
--------------------

and that that is what gas for ARM expects.


Chris, do you think it is OK?

> deep

Cheers,
-- 
Rafael Ávila de Espíndola
-------------- next part --------------
A non-text attachment was scrubbed...
Name: comm.patch
Type: text/x-diff
Size: 4706 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20100126/4aac86a3/attachment.patch>