[PATCH] D33205: ARM] Fix Neon vector type alignment to 64-bit

Renato Golin via llvm-commits llvm-commits at lists.llvm.org
Tue May 16 11:06:31 PDT 2017


On 16 May 2017 at 18:10, Stephen Hines <srhines at google.com> wrote:
> In that case, the implementation and later, the specification, were
> changed to satisfy a different set of requirements. Here, the implementation
> has existed for an even longer time, so I think it is a mistake to change
> this now. Does the ARM compiler do something different (or does GCC)? Last
> time I checked, ext_vectors couldn't be safely used across compilers for
> various other reasons beyond just alignment/padding.

FWIW, GCC 5.4 behaves like current clang, GCC 7.0 behaves like the
patch proposes, ie. follows the current document specification.

cheers,
--renato


More information about the llvm-commits mailing list