Re: [GCC][PATCH][mid-end] Optimize x * copysign (1.0, y) [Patch (1/2)]

2017-07-09 Thread Andrew Pinski
On Mon, Jun 12, 2017 at 12:56 AM, Tamar Christina wrote: > Hi All, > > this patch implements a optimization rewriting > > x * copysign (1.0, y) and > x * copysign (-1.0, y) > > to: > > x ^ (y & (1 << sign_bit_position)) > > This is done by creating a special builtin during matching and generate th

Re: [Patch, fortran] PR34640 - ICE when assigning item of a derived-component to a pointer

2017-07-09 Thread Thomas Koenig
Hi Paul, Please find attached what I believe is the final version of the patch. This is looking very good, this does work as advertised and has all the corner cases coverd. I also regtested this on powerpc64-unknown-linux-gnu. I took the opportunity of the delay, while the bounds issue was

Re: [PATCH] PR target/81313: Use DRAP only if there are outgoing arguments on stack

2017-07-09 Thread H.J. Lu
On Sun, Jul 9, 2017 at 11:57 AM, Uros Bizjak wrote: > On Sun, Jul 9, 2017 at 8:29 PM, H.J. Lu wrote: >> On Sun, Jul 9, 2017 at 11:19 AM, Uros Bizjak wrote: >>> On Fri, Jul 7, 2017 at 12:14 AM, H.J. Lu wrote: On Thu, Jul 6, 2017 at 12:08 PM, H.J. Lu wrote: > Since DRAP is needed only i

Re: [PATCH] PR target/81313: Use DRAP only if there are outgoing arguments on stack

2017-07-09 Thread Uros Bizjak
On Sun, Jul 9, 2017 at 9:06 PM, H.J. Lu wrote: >>> PR target/81313 >>> * gcc.dg/stack-layout-dynamic-1.c (dg-options): Add -mregparm=3 >>> for ia32. >>> Don't expect cfi_escape and expect cfi_def_cfa_register on x86. >>> >>> OK for trunk? >> >> Better use something like: >> >> --cut here-- >> Ind

Re: *Ping* [patch, libgfortran] Use memcpy in a few more places for eoshift

2017-07-09 Thread Thomas Koenig
Hi Paul, The patch is OK by me. Thanks for the review. Committed as rev. 250085. Thanks for working on speeding up these library functions. Does the octave version, mentioned in the clf thread, translate easily into C? I had to remind myself of how octave cell arrays function. It is certain

Re: [PATCH] PR target/81313: Use DRAP only if there are outgoing arguments on stack

2017-07-09 Thread H.J. Lu
On Sun, Jul 9, 2017 at 11:57 AM, Uros Bizjak wrote: > On Sun, Jul 9, 2017 at 8:29 PM, H.J. Lu wrote: >> On Sun, Jul 9, 2017 at 11:19 AM, Uros Bizjak wrote: >>> On Fri, Jul 7, 2017 at 12:14 AM, H.J. Lu wrote: On Thu, Jul 6, 2017 at 12:08 PM, H.J. Lu wrote: > Since DRAP is needed only i

[PATCH] Ignore random output from Asan tests in dg-cmp-results

2017-07-09 Thread Yuri Gribov
Hi, Currently dg-cmp-results fails to compare fails in Asan tests because their output includes failing process id: NA->FAIL: c-c++-common/asan/bitfield-1.c -O0 output pattern test, is ==18161==ERROR FAIL->NA: c-c++-common/asan/bitfield-1.c -O0 output pattern test, is ==26667==ERROR This pa

[PATCH] Improve extraction of changed file in contrib/mklog

2017-07-09 Thread Yuri Gribov
Hi, Currently mklog will fail to analyze lines like this in patches: diff -rupN gcc/gcc/testsuite/lib/profopt.exp gcc-compare-checks/gcc/testsuite/lib/profopt.exp (it fails with "Error: failed to parse diff for ... and ..."). This patch fixes it. Ok for trunk? -Y mklog-filename-fix-1.patch Des

[PATCH] Print relative names in UNSUPPORTED AutoFDO tests

2017-07-09 Thread Yuri Gribov
Hi, This patch fixes AutoFDO Dejagnu tests to print relative filenames instead of absolute ones when AutoFDO is unsupported. This is to prevent spurious differences in dg-cmp-results.sh, like this: NA->UNSUPPORTED: /home/yugr/src/gcc-57371/gcc/testsuite/g++.dg/bprob/g++-bprob-1.C -fauto-profile

Re: [PATCH] PR target/81313: Use DRAP only if there are outgoing arguments on stack

2017-07-09 Thread Uros Bizjak
On Sun, Jul 9, 2017 at 8:29 PM, H.J. Lu wrote: > On Sun, Jul 9, 2017 at 11:19 AM, Uros Bizjak wrote: >> On Fri, Jul 7, 2017 at 12:14 AM, H.J. Lu wrote: >>> On Thu, Jul 6, 2017 at 12:08 PM, H.J. Lu wrote: Since DRAP is needed only if there are outgoing arguments on stack, we should tra

Re: [Patch, fortran] PR34640 - ICE when assigning item of a derived-component to a pointer

2017-07-09 Thread Paul Richard Thomas
Hi Thomas, Hi All, Please find attached what I believe is the final version of the patch. The problem concerning temporaries being generated in lieu of the descriptor being passed directly - see pointer_array_7.f90 and the change to subref_array_4.f90. This latter necessitated a thread on clf to

Re: [PATCH] PR target/81313: Use DRAP only if there are outgoing arguments on stack

2017-07-09 Thread H.J. Lu
On Sun, Jul 9, 2017 at 11:19 AM, Uros Bizjak wrote: > On Fri, Jul 7, 2017 at 12:14 AM, H.J. Lu wrote: >> On Thu, Jul 6, 2017 at 12:08 PM, H.J. Lu wrote: >>> Since DRAP is needed only if there are outgoing arguments on stack, we >>> should track outgoing arguments on stack and avoid setting need_

Re: [PATCH] PR target/81313: Use DRAP only if there are outgoing arguments on stack

2017-07-09 Thread Uros Bizjak
On Fri, Jul 7, 2017 at 12:14 AM, H.J. Lu wrote: > On Thu, Jul 6, 2017 at 12:08 PM, H.J. Lu wrote: >> Since DRAP is needed only if there are outgoing arguments on stack, we >> should track outgoing arguments on stack and avoid setting need_drap to >> true when there are no outgoing arguments on st

Re: *Ping* [patch, libgfortran] Use memcpy in a few more places for eoshift

2017-07-09 Thread Paul Richard Thomas
Hi Thomas, The patch is OK by me. Thanks for working on speeding up these library functions. Does the octave version, mentioned in the clf thread, translate easily into C? I had to remind myself of how octave cell arrays function. It is certainly a remarkably concise solution. Cheers Paul On 8

[Patch Fortran committed] PR81341 - trunk/gcc/fortran/class.c:313: redundant condition ?

2017-07-09 Thread Dominique d'Humières
I have committed as obvious the following patch at revision r250083 --- ../_clean/gcc/fortran/class.c 2017-04-14 23:53:52.0 +0200 +++ gcc/fortran/class.c 2017-07-08 12:25:51.0 +0200 @@ -310,7 +310,6 @@ class_array_ref_detected (gfc_ref *ref, else if (ref->next && ref-

[PING][PATCH][PR sanitizer/77631] Support separate debug info in libbacktrace

2017-07-09 Thread Denis Khalikov
Hello, this is a ping for that patch https://gcc.gnu.org/ml/gcc-patches/2017-07/msg00022.html Thanks.

Re: *Ping* [patch, libgfortran] Use memcpy in a few more places for eoshift

2017-07-09 Thread Thomas Koenig
Am 08.07.2017 um 13:57 schrieb Thomas Koenig: Am 04.07.2017 um 00:06 schrieb Thomas Koenig: attached are a few more speedups for special eoshift cases. This time, nothing fancy, just use memcpy for copying in the contiguous case. Ping? Regards Thomas Some benchmarks (source attached