[PATCH] clang compatibility: use p2align

Volodymyr Turanskyy Volodymyr.Turanskyy@arm.com
Mon Jan 20 15:29:31 GMT 2025


Hi,

Please see attached re-formatted patch file.

Sincerely,

Volodymyr Turanskyy | Principal Software Engineer | CE SW LLVM Team

From: Joel Sherrill <joel.sherrill@gmail.com>
Date: Monday, 20 January 2025 at 14:58
To: Volodymyr Turanskyy <Volodymyr.Turanskyy@arm.com>
Cc: newlib@sourceware.org <newlib@sourceware.org>, nd <nd@arm.com>
Subject: Re: [PATCH] clang compatibility: use p2align
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 <Volodymyr.Turanskyy@arm.com<mailto:Volodymyr.Turanskyy@arm.com>> 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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://sourceware.org/pipermail/newlib/attachments/20250120/8152bdc5/attachment-0001.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Use-.p2align-2-instead-of-.align-0.patch
Type: application/octet-stream
Size: 2134 bytes
Desc: 0001-Use-.p2align-2-instead-of-.align-0.patch
URL: <https://sourceware.org/pipermail/newlib/attachments/20250120/8152bdc5/attachment-0001.obj>


More information about the Newlib mailing list