[PATCH] clang compatibility: use p2align

Richard Earnshaw (lists) Richard.Earnshaw@arm.com
Fri Feb 21 15:22:59 GMT 2025


On 20/01/2025 15:29, Volodymyr Turanskyy wrote:
> 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 <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 <https://sourceware.org/pipermail/newlib/2025/021452.html>
> 
>      
> 
>      
> 
>     Sincerely,
> 
>      
> 
>     *Volodymyr Turanskyy* | Principal Software Engineer | CE SW LLVM Team
> 

Pushed, thanks.

R.


More information about the Newlib mailing list