Re: [PATCH] clang compatibility: use p2align

Joel Sherrill <[email protected]>
Newsgroups gmane.comp.lib.newlib
Message-ID <CAF9ehCVBaKyBrhVj+9CMnNaUSBfbz_Na2KrNZzh3m=voc5Nj4g@mail.gmail.com>
The patch looks ok but it does not have a commit message.  git format-patch
should do the trick.

I have a meeting in 2 minutes and can't look at the other two right now.

--joel

On Mon, Jan 20, 2025 at 8:49 AM Volodymyr Turanskyy <
[email protected]> wrote:

> Hi,
>
>
>
> Following up on the RFC [1], please find attached a patch that uses
> p2aling since as per
> https://www.sourceware.org/binutils/docs/as/ARM-Directives.html GCC
> .align 0 is a special case that actually means .p2align 2 which is explicit
> and handled by clang.
>
>
>
> References
>
>    1. https://sourceware.org/pipermail/newlib/2025/021452.html
>
>
>
>
>
> Sincerely,
>
>
>
> *Volodymyr Turanskyy* | Principal Software Engineer | CE SW LLVM Team
>
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.