Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Tue, 25 May 2021 18:07:42 -0700
From: Joao Moreira <joao@...rdrivepizza.com>
To: Daniel Gutson <danielgutson@...il.com>, musl@...ts.openwall.com
Subject: Re: Intel CET Support



On 5/25/21 5:12 PM, Daniel Gutson wrote:
> 
> 
> El mar., 25 may. 2021 8:00 p. m., Joao Moreira <joao@...rdrivepizza.com 
> <mailto:joao@...rdrivepizza.com>> escribió:
> 
>     Hi,
> 
>     Long ago I sent some patches here to enable CET support within MUSL
>     (https://www.openwall.com/lists/musl/2020/10/19/3
>     <https://www.openwall.com/lists/musl/2020/10/19/3>).
> 
> 
> Did you intentionally remove the : in the label?
> If not, does this assembly without errors? How did you test this?
> 
> @@ -49,7 +52,8 @@ feclearexcept:
>   
>   .global feraiseexcept
>   .type feraiseexcept,@function
> -feraiseexcept:	
> +feraiseexcept
> +	ENDBRANCH32
> 

I just checked my local repo and it still has the ":" after the label. 
I'm assuming that I accidentally removed it when I was reading/reviewing 
the git-generated .patch files in vim before sending it (after these 
were compiled and tested).

Thanks for spotting it.

Powered by blists - more mailing lists

Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.