Re: [PATCH] fwprop: Fix single_use_p calculation

2021-03-15 Thread Stefan Liebler via Gcc-patches
On 02/03/2021 23:37, Ilya Leoshkevich wrote: > Bootstrapped and regtested on x86_64-redhat-linux, ppc64le-redhat-linux > and s390x-redhat-linux. Ok for master? > > > > Commit efb6bc55a93a ("fwprop: Allow (subreg (mem)) simplifications") > introduced a check that was supposed to look at the prop

Re: [PATCH] S/390: Fix PR91628

2020-04-07 Thread Stefan Liebler via Gcc-patches
On 4/1/20 6:20 PM, Stefan Liebler wrote: On 4/1/20 12:50 PM, Iain Buclaw wrote: On 01/04/2020 08:28, Stefan Liebler wrote: ping Hi Stefan, As I've already said, I think that the name should be __ibmz_get_tls_offset to make clear that it is an internal function. Other than that, looks g

Re: [PATCH] S/390: Fix PR91628

2020-04-01 Thread Stefan Liebler via Gcc-patches
On 4/1/20 12:50 PM, Iain Buclaw wrote: On 01/04/2020 08:28, Stefan Liebler wrote: ping Hi Stefan, As I've already said, I think that the name should be __ibmz_get_tls_offset to make clear that it is an internal function. Other than that, looks good to me. Iain. Hi Iain, Sorry. I've

Re: [PATCH] S/390: Fix layout of struct sigaction_t

2020-04-01 Thread Stefan Liebler via Gcc-patches
On 4/1/20 12:54 PM, Iain Buclaw wrote: On 01/04/2020 08:28, Stefan Liebler wrote: ping Thanks, I'll send the patch upstream, as it's the same there. Looks OK to me. Regards Iain. Thanks for committing the patch upstream

Re: [PATCH] S/390: Fix PR91628

2020-03-31 Thread Stefan Liebler via Gcc-patches
ping On 3/23/20 6:04 PM, Stefan Liebler wrote: Hi, this patch picks up Robin Dapps patch __tls_get_offset-in-separate.S. See "Bugzilla 91628 - libdruntime uses glibc internal symbol on s390" (https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91628) The original purpose was to get rid of the usage o

Re: [PATCH] S/390: Fix layout of struct sigaction_t

2020-03-31 Thread Stefan Liebler via Gcc-patches
ping On 3/23/20 6:05 PM, Stefan Liebler wrote: Hi, the ordering of some fields inĀ  struct sigaction on s390x (64bit) differs compared to s390 and other architectures. This patch adjusts this order according to the definition of /sysdeps/unix/sysv/linux/s390/bits/sigaction.h Without this fix e.

[PATCH] S/390: Fix layout of struct sigaction_t

2020-03-23 Thread Stefan Liebler via Gcc-patches
Hi, the ordering of some fields in struct sigaction on s390x (64bit) differs compared to s390 and other architectures. This patch adjusts this order according to the definition of /sysdeps/unix/sysv/linux/s390/bits/sigaction.h Without this fix e.g. the call sigaction( suspendSignalNumber, &sigu

[PATCH] S/390: Fix PR91628

2020-03-23 Thread Stefan Liebler via Gcc-patches
Hi, this patch picks up Robin Dapps patch __tls_get_offset-in-separate.S. See "Bugzilla 91628 - libdruntime uses glibc internal symbol on s390" (https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91628) The original purpose was to get rid of the usage of the glibc-internal symbol __tls_get_addr_inter