Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAPhv3nof+aodqqwwnX8aqGCcEf83y9AK0exxmWLNGCs3Wa+1+w@mail.gmail.com>
Date: Wed, 22 Jan 2020 14:31:25 +0000
From: Luís Marques <luismarques@...risc.org>
To: musl@...ts.openwall.com
Subject: Re: [PATCH] Fix RISC-V a_cas inline asm operand sign extension

On Wed, Jan 15, 2020 at 1:33 PM Luís Marques <luismarques@...risc.org> wrote:
> This patch adds an explicit cast to the int arguments passed to the inline asm
> used in the RISC-V's implementation of `a_cas`, to ensure that they are properly
> sign extended to 64 bits. They aren't automatically sign extended by Clang, and
> GCC technically also doesn't guarantee that they will be sign extended.

Does anyone have any feedback regarding this patch?
If not, perhaps it could be merged?

Best regards,
Luís Marques

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.