[PATCH] fix alignment specifier syntax on Neon instructions

Renato Golin renato.golin at linaro.org
Tue Feb 12 09:04:04 PST 2013


On 12 February 2013 15:56, Kristof Beyls <kristof.beyls at arm.com> wrote:

> The background to these patches is that currently, LLVM expects the
> alignment specifier in some Neon instructions to be specified using ", :"
>

Hi Kristof,

The patch looks good to me, but I don't know about Darwin assembler to give
you a good answer on that.

Binutils 2.20 is reasonably old, but already GPLv3 (changed in 2.18), what
could cause Darwin not to take it as is.

I agree that the first patch is safe, but I also think that we should
output the correct syntax as well. If this is a problem for Darwin, I'd
recommend creating an option default to CORRECT and set to OLD by the
Darwin Target builder.

cheers,
--renato
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130212/36be7964/attachment.html>


More information about the llvm-commits mailing list