Wednesday, October 30, 2024 Richard Sandiford <richard.sandif...@arm.com> wrote:
> So for the change in question, I was expecting, say, a 1024-byte-aligned > float[3] to be defined using: > > .lcomm array, 1024 Thank you for noticing this. The patch series has been rebased from the upstream before reproducing the assembly code. It looks like there were recent changes in the upstream and __attribute__((aligned(32))) works as expected without LTO, and FFmpeg can be compiled and pass the test. The patch is still needed for LTO, however it makes sense to understand how the same approach with alignment can be used also in LTO. For now, the patch can be skipped. Thank you for the helpful review. Regards, Evgeny
