* [PATCH 3/4, libitm, sh]: Change gtm_futex_{wait,wake} to int
@ 2015-05-27 22:41 Uros Bizjak
2015-05-28 1:27 ` Kaz Kojima
0 siblings, 1 reply; 3+ messages in thread
From: Uros Bizjak @ 2015-05-27 22:41 UTC (permalink / raw)
To: gcc-patches; +Cc: Richard Henderson
2015-05-27 Uros Bizjak <ubizjak@gmail.com>
* config/linux/sh/futex_bits.h (sys_futex0) Change operands
"op" and "val" to int.
Untested.
OK for mainline?
Uros.
Index: config/linux/sh/futex_bits.h
===================================================================
--- config/linux/sh/futex_bits.h (revision 223771)
+++ config/linux/sh/futex_bits.h (working copy)
@@ -32,7 +32,7 @@
trapa #0x14; or r0,r0; or r0,r0; or r0,r0; or r0,r0; or r0,r0"
static inline long
-sys_futex0 (std::atomic<int> *addr, long op, long val)
+sys_futex0 (std::atomic<int> *addr, int op, int val)
{
int __status;
register long __r3 asm ("r3") = SYS_futex;
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 3/4, libitm, sh]: Change gtm_futex_{wait,wake} to int
2015-05-27 22:41 [PATCH 3/4, libitm, sh]: Change gtm_futex_{wait,wake} to int Uros Bizjak
@ 2015-05-28 1:27 ` Kaz Kojima
2015-05-28 18:25 ` Uros Bizjak
0 siblings, 1 reply; 3+ messages in thread
From: Kaz Kojima @ 2015-05-28 1:27 UTC (permalink / raw)
To: ubizjak; +Cc: gcc-patches, rth
Uros Bizjak <ubizjak@gmail.com> wrote:
> 2015-05-27 Uros Bizjak <ubizjak@gmail.com>
>
> * config/linux/sh/futex_bits.h (sys_futex0) Change operands
> "op" and "val" to int.
>
> Untested.
>
> OK for mainline?
OK.
Although it looks obvious, I've confirmed that there is no build
issue/regression on sh4-unknown-linux-gnu with it and the patch 1/4.
Regards,
kaz
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 3/4, libitm, sh]: Change gtm_futex_{wait,wake} to int
2015-05-28 1:27 ` Kaz Kojima
@ 2015-05-28 18:25 ` Uros Bizjak
0 siblings, 0 replies; 3+ messages in thread
From: Uros Bizjak @ 2015-05-28 18:25 UTC (permalink / raw)
To: Kaz Kojima; +Cc: gcc-patches, Richard Henderson
On Thu, May 28, 2015 at 12:52 AM, Kaz Kojima <kkojima@rr.iij4u.or.jp> wrote:
>> * config/linux/sh/futex_bits.h (sys_futex0) Change operands
>> "op" and "val" to int.
>>
>> Untested.
>>
>> OK for mainline?
>
> OK.
>
> Although it looks obvious, I've confirmed that there is no build
> issue/regression on sh4-unknown-linux-gnu with it and the patch 1/4.
Thanks! I went ahead and commit the whole patch series to mainline SVN.
Uros.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2015-05-28 17:38 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-05-27 22:41 [PATCH 3/4, libitm, sh]: Change gtm_futex_{wait,wake} to int Uros Bizjak
2015-05-28 1:27 ` Kaz Kojima
2015-05-28 18:25 ` Uros Bizjak
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).