Re: Ping^3 : [PATCH] [gcc, combine] PR46164: Don't combine the insns if a volatile register is contained.

2015-04-22 Thread Terry Guo
On Wed, Apr 22, 2015 at 10:30 AM, Segher Boessenkool wrote: > On Wed, Apr 22, 2015 at 10:21:43AM +0800, Terry Guo wrote: >> gcc/ChangeLog: >> 2015-04-22 Hale Wang >> Terry Guo >> >>PR rtl-optimization/64818 >>* combine.c (can_combine_p): Don't combine user-sp

Re: Expand oacc kernels after pass_fre (was: [PATCH, 1/8] Expand oacc kernels after pass_build_ealias)

2015-04-22 Thread Richard Biener
On Tue, 21 Apr 2015, Thomas Schwinge wrote: > Hi! > > On Tue, 25 Nov 2014 12:22:02 +0100, Tom de Vries > wrote: > > On 24-11-14 11:56, Tom de Vries wrote: > > > On 15-11-14 18:19, Tom de Vries wrote: > > >> On 15-11-14 13:14, Tom de Vries wrote: > > >>> I'm submitting a patch series with initia

Re: [PATCH, 3/8] Add pass_ch_oacc_kernels to pass_oacc_kernels

2015-04-22 Thread Richard Biener
On Tue, 21 Apr 2015, Thomas Schwinge wrote: > Hi! > > On Tue, 25 Nov 2014 12:27:34 +0100, Tom de Vries > wrote: > > On 15-11-14 18:21, Tom de Vries wrote: > > > On 15-11-14 13:14, Tom de Vries wrote: > > >> Hi, > > >> > > >> I'm submitting a patch series with initial support for the oacc kernel

Re: [PATCH, 4/8] Add pass_tree_loop_{init,done} to pass_oacc_kernels

2015-04-22 Thread Richard Biener
On Tue, 21 Apr 2015, Thomas Schwinge wrote: > Hi! > > On Tue, 25 Nov 2014 12:29:28 +0100, Tom de Vries > wrote: > > On 15-11-14 18:21, Tom de Vries wrote: > > > On 15-11-14 13:14, Tom de Vries wrote: > > >> I'm submitting a patch series with initial support for the oacc kernels > > >> directive

[PATCH][PR65823] Fix va_arg ap_copy nop detection

2015-04-22 Thread Tom de Vries
Hi, this patch fixes PR65823. The problem is a verify_gimple ICE during compilation of gcc.c-torture/execute/stdarg-2.c for arm at -O0/-O1: ... In function 'f3': src/gcc/testsuite/gcc.c-torture/execute/stdarg-2.c:61:1: error: incorrect sharing of tree nodes aps[4] # .MEM_5 = VDEF <.MEM_11> aps

Re: [PATCH, 6/8] Add pass_copy_prop in pass_oacc_kernels

2015-04-22 Thread Richard Biener
On Tue, 21 Apr 2015, Thomas Schwinge wrote: > Hi! > > On Tue, 25 Nov 2014 12:38:55 +0100, Tom de Vries > wrote: > > On 15-11-14 18:22, Tom de Vries wrote: > > > On 15-11-14 13:14, Tom de Vries wrote: > > >> I'm submitting a patch series with initial support for the oacc kernels > > >> directive

Re: [PATCH] [1/2] [ARM] [libgcc] Support RTABI half-precision conversion functions.

2015-04-22 Thread Ramana Radhakrishnan
On Mon, Apr 13, 2015 at 12:25 PM, Joseph Myers wrote: > On Mon, 13 Apr 2015, Hale Wang wrote: > >> Yes, you are right. It's my fault to add the "only" here. Thank you to point >> out this. >> Beside this, is this patch OK for you? > > I don't think it's a good idea for libgcc to include large piec

Re: [wwwdocs] Add libstdc++ ABI changes to /gcc-5/changes.html

2015-04-22 Thread Jonathan Wakely
On 21 April 2015 at 12:54, I wrote: > I plan to commit this to wwwdocs later today, it adds a caveat to the > top of the file, with a link to a larger description in the libstdc++ > section, which links to the new page I've just added to the manual. I committed it, and fixed the invalid HTML in 0.

Re: [PATCH][PR65823] Fix va_arg ap_copy nop detection

2015-04-22 Thread Richard Biener
On Wed, Apr 22, 2015 at 9:41 AM, Tom de Vries wrote: > Hi, > > this patch fixes PR65823. > > The problem is a verify_gimple ICE during compilation of > gcc.c-torture/execute/stdarg-2.c for arm at -O0/-O1: > ... > In function 'f3': > src/gcc/testsuite/gcc.c-torture/execute/stdarg-2.c:61:1: > error:

Re: [PATCH, bfin] handle BFIN_CPU_UNKNOWN in TARGET_CPU_CPP_BUILTINS

2015-04-22 Thread Bernhard Reutner-Fischer
On 8 April 2015 at 22:12, Bernhard Reutner-Fischer wrote: > ping This was OK'ed by Jeff and committed to trunk as r222307. thanks, > > On 1 April 2015 at 23:34, Bernhard Reutner-Fischer > wrote: >> Hi, >> >> gcc/c-family/c-cppbuiltin.c >> In file included from ./tm.h:21:0, >> fr

Re: [PATCH, c6x] handle unk_isa in TARGET_CPU_CPP_BUILTINS

2015-04-22 Thread Bernhard Reutner-Fischer
On 8 April 2015 at 22:13, Bernhard Reutner-Fischer wrote: > ping This was OKed by Jeff and committed to trunk as r222308. thanks, > > On 2 April 2015 at 22:49, Bernhard Reutner-Fischer > wrote: >> On Wed, Apr 01, 2015 at 11:37:41PM +0200, Bernhard Reutner-Fischer wrote: >>>Bernd, >>> >>>same for

Re: [PATCH] emit_bss(): Remove redundant guard

2015-04-22 Thread Bernhard Reutner-Fischer
On 8 April 2015 at 20:19, Bernhard Reutner-Fischer wrote: > gcc/ChangeLog: OKed by Jeff and installed to trunk as r222311. thanks, > > 2015-04-01 Bernhard Reutner-Fischer > > * varasm.c (emit_bss): Remove redundant guard. > > Signed-off-by: Bernhard Reutner-Fischer > --- > gcc/varasm

Re: [PATCH 2/2] tree-tailcall: Commentary typo fix, remove fwd declaration

2015-04-22 Thread Bernhard Reutner-Fischer
On 8 April 2015 at 20:32, Bernhard Reutner-Fischer wrote: > gcc/ChangeLog: OKed by Jeff and committed to trunk as r222312. thanks, > > 2015-04-01 Bernhard Reutner-Fischer > > * tree-tailcall.c (suitable_for_tail_opt_p, find_tail_calls): > Remove unneeded forward declarations.

Re: [PATCH] PR target/55144: bfin: fix opening glibc-c.o: No such file or directory

2015-04-22 Thread Bernhard Reutner-Fischer
On 8 April 2015 at 20:37, Bernhard Reutner-Fischer wrote: > building all-gcc for bfin-linux-uclibc results in > > build/genchecksum cp/cp-lang.o c-family/stub-objc.o ... glibc-c.o \ > libbackend.a .. > cc1plus-checksum.c.tmp > opening glibc-c.o: No such file or directory > make[2]: *** [cc1-checks

Re: [PATCH] Optionally sanitize globals in user-defined sections

2015-04-22 Thread Yury Gribov
On 04/19/2015 06:11 PM, Jakub Jelinek wrote: On Sun, Apr 19, 2015 at 10:54:57AM +0300, Yury Gribov wrote: On 04/17/2015 08:29 PM, Andi Kleen wrote: Yury Gribov writes: + +static bool +section_sanitized_p (const char *sec) +{ + if (!sanitized_sections) +return false; + size_t len = strle

RE: [PATCH] [1/2] [ARM] [libgcc] Support RTABI half-precision conversion functions.

2015-04-22 Thread Hale Wang
> -Original Message- > From: Ramana Radhakrishnan [mailto:ramana@googlemail.com] > Sent: Wednesday, April 22, 2015 3:50 PM > To: Joseph Myers > Cc: Hale Wang; GCC Patches > Subject: Re: [PATCH] [1/2] [ARM] [libgcc] Support RTABI half-precision > conversion functions. > > On Mon, Apr 13

[PATCH] PR target/47122 vax-*-openbsd* config.gcc typo

2015-04-22 Thread Bernhard Reutner-Fischer
Gerald / jsg typoed this in the initial r152218 a.k.a 8e6600a15374f9446d5026e8ef6b68e0d753fb6c gcc/ChangeLog: 2015-04-09 Bernhard Reutner-Fischer PR target/47122 * config.gcc (vax-*-openbsd*): Fix name of pthread spec header. --- gcc/config.gcc |2 +- 1 file changed, 1 in

Re: [PATCH] PR target/47098 OBSD_LIB_SPEC on i686-openbsd3 missing

2015-04-22 Thread Bernhard Reutner-Fischer
On 13 April 2015 at 15:08, Jeff Law wrote: > On 04/11/2015 04:15 PM, Bernhard Reutner-Fischer wrote: >> >> gcc/config/openbsd.h:143:18: error: ‘OBSD_LIB_SPEC’ was not declared in >> this scope >> #define LIB_SPEC OBSD_LIB_SPEC >>^ >> gcc/gcc.c:879:31: note: >> in expansion of

[PATCH ARM]Fix pr42172-1.c failure on pre armv7 processors

2015-04-22 Thread Bin Cheng
Hi, Case pr42172-1.c failed on pre-armv7 processors because GCC actually generates better code without ldr instruction. This patch just refines test case by checking str instead of ldr, makes sure the case passes on all arm processors. In the end, we need to fix GCC combiner to generate optimal c

Re: [PATCH] Optionally sanitize globals in user-defined sections

2015-04-22 Thread Yury Gribov
On 04/22/2015 11:31 AM, Yury Gribov wrote: On 04/19/2015 06:11 PM, Jakub Jelinek wrote: On Sun, Apr 19, 2015 at 10:54:57AM +0300, Yury Gribov wrote: On 04/17/2015 08:29 PM, Andi Kleen wrote: Yury Gribov writes: + +static bool +section_sanitized_p (const char *sec) +{ + if (!sanitized_sectio

Re: [PATCH] PR target/47122 vax-*-openbsd* config.gcc typo

2015-04-22 Thread Bernhard Reutner-Fischer
I've applied this as r222314. thanks, On 22 April 2015 at 10:36, Bernhard Reutner-Fischer wrote: > Gerald / jsg typoed this in the initial r152218 a.k.a > 8e6600a15374f9446d5026e8ef6b68e0d753fb6c > > gcc/ChangeLog: > > 2015-04-09 Bernhard Reutner-Fischer > > PR target/47122 >

Re: [C/C++ PATCH] Improve -Wlogical-op (PR c/63357)

2015-04-22 Thread Marek Polacek
On Tue, Apr 21, 2015 at 04:19:26PM +0200, Manuel López-Ibáñez wrote: > On 21/04/15 13:16, Marek Polacek wrote: > >(-Wlogical-op still isn't enabled neither by -Wall nor by -Wextra.) > > The reason is https://gcc.gnu.org/PR61534 > > which means we don't want to warn for: > > extern int xxx; > #de

Re: [PATCH] [1/2] [ARM] [libgcc] Support RTABI half-precision conversion functions.

2015-04-22 Thread Ramana Radhakrishnan
On Wed, Apr 22, 2015 at 9:32 AM, Hale Wang wrote: >> -Original Message- >> From: Ramana Radhakrishnan [mailto:ramana@googlemail.com] >> Sent: Wednesday, April 22, 2015 3:50 PM >> To: Joseph Myers >> Cc: Hale Wang; GCC Patches >> Subject: Re: [PATCH] [1/2] [ARM] [libgcc] Support RTABI h

Re: [PATCH] Optionally sanitize globals in user-defined sections

2015-04-22 Thread Jakub Jelinek
On Wed, Apr 22, 2015 at 11:43:53AM +0300, Yury Gribov wrote: > @@ -272,7 +273,7 @@ along with GCC; see the file COPYING3. If not see > > static unsigned HOST_WIDE_INT asan_shadow_offset_value; > static bool asan_shadow_offset_computed; > -static const char *sanitized_sections; > +static vec *s

Re: [PATCH] Optionally sanitize globals in user-defined sections

2015-04-22 Thread Yury Gribov
On 04/22/2015 12:00 PM, Jakub Jelinek wrote: On Wed, Apr 22, 2015 at 11:43:53AM +0300, Yury Gribov wrote: @@ -272,7 +273,7 @@ along with GCC; see the file COPYING3. If not see static unsigned HOST_WIDE_INT asan_shadow_offset_value; static bool asan_shadow_offset_computed; -static const cha

Re: [PATCH] Optionally sanitize globals in user-defined sections

2015-04-22 Thread Jakub Jelinek
On Wed, Apr 22, 2015 at 12:26:57PM +0300, Yury Gribov wrote: > On 04/22/2015 12:00 PM, Jakub Jelinek wrote: > >On Wed, Apr 22, 2015 at 11:43:53AM +0300, Yury Gribov wrote: > >>@@ -272,7 +273,7 @@ along with GCC; see the file COPYING3. If not see > >> > >> static unsigned HOST_WIDE_INT asan_shadow

Re: [C/C++ PATCH] Improve -Wlogical-op (PR c/63357)

2015-04-22 Thread Manuel López-Ibáñez
On 22 April 2015 at 10:59, Marek Polacek wrote: > So the bug really must be solved first :(. I see I proposed a solution in the > PR, I wonder whether that works... It will work for extern int xxx; #define XXX !xxx int test (void) { if (XXX && xxx) return 4; else return 0; } becaus

Re: [C/C++ PATCH] Improve -Wlogical-op (PR c/63357)

2015-04-22 Thread Manuel López-Ibáñez
On 22 April 2015 at 11:50, Manuel López-Ibáñez wrote: > So, I think your proposal is an improvement, however, it does not > fully fix the problem. For that, we need to have locations for the > operands of expressions (PR43486), so we can check if they are also > come from macro expansion. Or, in

[gomp4] Merge trunk r221638 (2015-03-24) into gomp-4_0-branch

2015-04-22 Thread Thomas Schwinge
Hi! Committed to gomp-4_0-branch in r222320: commit d9af23c3cb81d769ce6ca32c26b147ebafe9628c Merge: 7109b39 5622376 Author: tschwinge Date: Wed Apr 22 10:07:47 2015 + svn merge -r 221363:221638 svn+ssh://gcc.gnu.org/svn/gcc/trunk git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc

Re: [PATCH][libstc++v3]Add new dg-require-thread-fence directive.

2015-04-22 Thread Jonathan Wakely
On 21/04/15 15:29 +0100, Renlin Li wrote: Hi all, This patch defines a new dg-require-thread-fence directive. And three test cases are updated to use it. The new directive are used to check whether the target support thread fence either by the target back-end or external library function cal

[PATCH, FRV, libgcc]: Move CRT_GET_RFIB_DATA to libgcc

2015-04-22 Thread Uros Bizjak
Hello! Attached patch moves CRT_GET_RFIB_DATA from gcc to libgcc in the same way as was recently done for x86 targets [1]. The second patch is needed to successfully compile frv-elf libgcc, otherwise libgcc build fails in defaults.h, line 499 with "Unknown BITS_PER_UNIT" error. libgcc/ChangeLog:

Re: [C/C++ PATCH] Improve -Wlogical-op (PR c/63357)

2015-04-22 Thread Marek Polacek
On Wed, Apr 22, 2015 at 11:50:17AM +0200, Manuel López-Ibáñez wrote: > So, I think your proposal is an improvement, however, it does not > fully fix the problem. For that, we need to have locations for the > operands of expressions (PR43486), so we can check if they are also > come from macro expan

Re: [PATCH][libstc++v3]Add new dg-require-thread-fence directive.

2015-04-22 Thread Renlin Li
Hi Jonathan, Thank you for the suggestion. I have just attached the updated the patch. It works as before. Is this Okay to commit? Regards, Renlin Li On 22/04/15 11:19, Jonathan Wakely wrote: On 21/04/15 15:29 +0100, Renlin Li wrote: Hi all, This patch defines a new dg-require-thread-fence

Re: [PATCH][AArch64] Implement -m{cpu,tune,arch}=native using only /proc/cpuinfo

2015-04-22 Thread Kyrill Tkachov
[Sorry for resending twice. My mail client glitched] On 20/04/15 16:47, Kyrill Tkachov wrote: Hi all, This is an attempt to add native CPU detection to AArch64 GNU/Linux targets. Similar to other ports we use SPEC rewriting to rewrite -m{cpu,tune,arch}=native options into the appropriate CPU/ar

Re: [PATCH][expr.c] PR 65358 Avoid clobbering partial argument during sibcall

2015-04-22 Thread Kyrill Tkachov
On 21/04/15 18:33, Kyrill Tkachov wrote: On 21/04/15 15:09, Jeff Law wrote: On 04/21/2015 02:30 AM, Kyrill Tkachov wrote: From reading config/stormy16/stormy-abi it seems to me that we don't pass arguments partially in stormy16, so this code would never be called there. That leaves pa as th

trunk test result inconsistencies

2015-04-22 Thread Andrew MacLeod
Is anyone else seeing comparison problems on trunk? I was having problems testing a patch on a 4/16 extraction, so last night I checked out a fresh trunk, built it, ran make check... then removed the build directory, re-built it from scratch again. make check.. and get a bunch of different re

Re: Add DSE to early passes

2015-04-22 Thread H.J. Lu
On Mon, Apr 13, 2015 at 3:01 AM, Jan Hubicka wrote: > Hi, > this patch adds DSE to early optimizations and handles testuiste fallout. > As discussed in PR 65076 this reduces number of CLOBBER statements in tramp3d > to 50% (and those accounts 29% of all code previously). The pass also quite > ofte

Re: Add DSE to early passes

2015-04-22 Thread Jan Hubicka
> > @@ -42,6 +42,6 @@ bool f(I a, I b, I c, I d) { > > // This works only if everything is inlined into 'f'. > > > > // { dg-final { scan-tree-dump-times ";; Function" 1 "fre2" } } > > -// { dg-final { scan-tree-dump-times "free" 19 "fre2" } } > > +// { dg-final { scan-tree-dump-times "free" 18 "

Re: trunk test result inconsistencies

2015-04-22 Thread Jakub Jelinek
On Wed, Apr 22, 2015 at 08:04:03AM -0400, Andrew MacLeod wrote: > Is anyone else seeing comparison problems on trunk? > > I was having problems testing a patch on a 4/16 extraction, so last night I > checked out a fresh trunk, built it, ran make check... then removed the > build directory, re-buil

Re: Add DSE to early passes

2015-04-22 Thread Richard Biener
On Wed, 22 Apr 2015, Jan Hubicka wrote: > > > @@ -42,6 +42,6 @@ bool f(I a, I b, I c, I d) { > > > // This works only if everything is inlined into 'f'. > > > > > > // { dg-final { scan-tree-dump-times ";; Function" 1 "fre2" } } > > > -// { dg-final { scan-tree-dump-times "free" 19 "fre2" } } >

Re: trunk test result inconsistencies

2015-04-22 Thread Andrew MacLeod
On 04/22/2015 08:19 AM, Jakub Jelinek wrote: On Wed, Apr 22, 2015 at 08:04:03AM -0400, Andrew MacLeod wrote: Is anyone else seeing comparison problems on trunk? I was having problems testing a patch on a 4/16 extraction, so last night I checked out a fresh trunk, built it, ran make check... the

Re: trunk test result inconsistencies

2015-04-22 Thread Jeff Law
On 04/22/2015 06:28 AM, Andrew MacLeod wrote: On 04/22/2015 08:19 AM, Jakub Jelinek wrote: On Wed, Apr 22, 2015 at 08:04:03AM -0400, Andrew MacLeod wrote: Is anyone else seeing comparison problems on trunk? I was having problems testing a patch on a 4/16 extraction, so last night I checked out

Re: trunk test result inconsistencies

2015-04-22 Thread H.J. Lu
On Wed, Apr 22, 2015 at 5:19 AM, Jakub Jelinek wrote: > On Wed, Apr 22, 2015 at 08:04:03AM -0400, Andrew MacLeod wrote: >> Is anyone else seeing comparison problems on trunk? >> >> I was having problems testing a patch on a 4/16 extraction, so last night I >> checked out a fresh trunk, built it, r

Re: [PATCH][libstc++v3]Add new dg-require-thread-fence directive.

2015-04-22 Thread Jonathan Wakely
On 22 April 2015 at 12:25, Renlin Li wrote: > Hi Jonathan, > > Thank you for the suggestion. I have just attached the updated the patch. It > works as before. > Is this Okay to commit? OK, thanks.

Re: [PATCH 5/5] libcc1: 'set debug compile': Display absolute GCC driver filename

2015-04-22 Thread Jeff Law
On 04/21/2015 03:41 PM, Jan Kratochvil wrote: Hi, with the patches so far after (gdb) set debug compile 1 one would get: searching for compiler matching regex ^(x86_64|i.86)(-[^-]*)?-linux(-gnu)?-gcc$ found compiler x86_64-unknown-linux-gnu-gcc But I believe it is more r

Re: [PATCH 4/5] libcc1: Add 'set compile-gcc'

2015-04-22 Thread Jeff Law
On 04/21/2015 03:41 PM, Jan Kratochvil wrote: as discussed in How to use compile & execute function in GDB https://sourceware.org/ml/gdb/2015-04/msg00026.html GDB currently searches for /usr/bin/ARCH-OS-gcc and chooses one but one cannot override which one. GDB would provide new

Re: trunk test result inconsistencies

2015-04-22 Thread Jakub Jelinek
On Wed, Apr 22, 2015 at 05:40:07AM -0700, H.J. Lu wrote: > On Wed, Apr 22, 2015 at 5:19 AM, Jakub Jelinek wrote: > > On Wed, Apr 22, 2015 at 08:04:03AM -0400, Andrew MacLeod wrote: > >> Is anyone else seeing comparison problems on trunk? > >> > >> I was having problems testing a patch on a 4/16 ex

Re: [PATCH 3/5] libcc1: set debug compile: Display GCC driver filename

2015-04-22 Thread Jeff Law
On 04/21/2015 03:41 PM, Jan Kratochvil wrote: Hi, as discussed in How to use compile & execute function in GDB https://sourceware.org/ml/gdb/2015-04/msg00026.html GDB currently searches for /usr/bin/ARCH-OS-gcc and chooses one but it does not display which one. It cannot, GCC m

Re: [PATCH 2/5] libcc1: Use libcc1.so.0->libcc1.so.1

2015-04-22 Thread Jeff Law
On 04/21/2015 03:41 PM, Jan Kratochvil wrote: Hi, see [patch 1/5], particularly: (3) Currently there is no backward or forward compatibility although there could be one implemented. Personally I think the 'compile' feature is still in experimental stage so that it is OK to require las

[PATCH] [AArch32] Additional bics patterns.

2015-04-22 Thread Alex Velenko
Hi, This patch adds arm rtl patterns to generate bics instructions with shift. Done full regression run on arm-none-eabi. Is patch ok? gcc/config 2015-04-22 Alex Velenko * arm/arm.md (andsi_not_shiftsi_si_scc): New pattern. * (andsi_not_shiftsi_si_scc_no_reuse): New pattern. gcc/tests

Re: [PATCH 1/5] libcc1: Make libcc1.so->libcc1.so.0

2015-04-22 Thread Jeff Law
On 04/21/2015 03:41 PM, Jan Kratochvil wrote: Hi, the next [patch 3/5] will change the libcc1.so API. I am not sure if the API change gets approved that way but for such case: (1) We really need to change GCC_FE_VERSION_0 -> GCC_FE_VERSION_1, this feature is there for this purpose. That i

Re: [PATCH 2/13] musl libc config

2015-04-22 Thread Jeff Law
On 04/20/2015 12:52 PM, Szabolcs Nagy wrote: Add musl libc support to gcc and the command line option -mmusl following other libc support code. Note that -m cannot be entirely correct: there are build time decisions based on the default libc. gcc/Changelog: 2015-04-16 Gregor Richards

Re: trunk test result inconsistencies

2015-04-22 Thread H.J. Lu
On Wed, Apr 22, 2015 at 6:10 AM, Jakub Jelinek wrote: > On Wed, Apr 22, 2015 at 05:40:07AM -0700, H.J. Lu wrote: >> On Wed, Apr 22, 2015 at 5:19 AM, Jakub Jelinek wrote: >> > On Wed, Apr 22, 2015 at 08:04:03AM -0400, Andrew MacLeod wrote: >> >> Is anyone else seeing comparison problems on trunk?

Re: [PATCH 12/13] libstdc++, libgfortran gthr workaround for musl

2015-04-22 Thread Jeff Law
On 04/20/2015 12:59 PM, Szabolcs Nagy wrote: libgcc/gthr-posix.h uses weak reference logic to determine if libpthread is linked into the application or not. This is broken unless there is special workaround with libc internal knowledge and even then static linking needs further manual link time

Re: [PATCH 11/13] unwind fix for musl

2015-04-22 Thread Jeff Law
On 04/20/2015 12:58 PM, Szabolcs Nagy wrote: dl_iterate_phdr depends on USE_PT_GNU_EH_FRAME. I think USE_PT_GNU_EH_FRAME could be enabled more generally (whenever libc provides dl_iterate_phdr), but I only made a conservative change. libgcc/Changelog: 2015-04-16 Gregor Richards

RE: [PATCH] [AArch32] Additional bics patterns.

2015-04-22 Thread Kyrill Tkachov
Hi Alex, On 22/04/15 14:12, Alex Velenko wrote: > Hi, > > This patch adds arm rtl patterns to generate bics instructions with shift. > > Done full regression run on arm-none-eabi. A bootstrap on arm-none-linux-gnueabihf would be nice too. > > > Is patch ok? > > gcc/config > > 2015-04-22 Alex Ve

Re: trunk test result inconsistencies

2015-04-22 Thread Jakub Jelinek
On Wed, Apr 22, 2015 at 06:16:24AM -0700, H.J. Lu wrote: > > I admit I have not tried, but am certainly not seeing Peter's fix > > in https://www.kernel.org/pub/linux/kernel/v3.x/ChangeLog-3.19.5 > > has it been fixed differently? Not seeing anything in the Fedora kernel > > package %changelog eit

Re: [PATCH][PR65823] Fix va_arg ap_copy nop detection

2015-04-22 Thread Tom de Vries
On 22-04-15 10:06, Richard Biener wrote: On Wed, Apr 22, 2015 at 9:41 AM, Tom de Vries wrote: Hi, this patch fixes PR65823. The patches fixes the problem by using operand_equal_p to do the equality test. Bootstrapped and reg-tested on x86_64. Did minimal non-bootstrap build on arm an

Re: [PATCH, rs6000, testsuite] Fix PR target/64579, __TM_end __builtin_tend failed to return transactional state

2015-04-22 Thread Peter Bergner
On Tue, 2015-04-21 at 21:17 -0500, Segher Boessenkool wrote: > On Tue, Apr 21, 2015 at 03:56:18PM -0500, Peter Bergner wrote: > > This patch also fixes some issues I hit with the tabortdc[i] and > > htm_m[ft]spr_ patterns when used with -m32 -mpowerpc64. > > Running the testsuite, or did you actua

Re: trunk test result inconsistencies

2015-04-22 Thread Andrew MacLeod
On 04/22/2015 08:33 AM, Jeff Law wrote: On 04/22/2015 06:28 AM, Andrew MacLeod wrote: On 04/22/2015 08:19 AM, Jakub Jelinek wrote: On Wed, Apr 22, 2015 at 08:04:03AM -0400, Andrew MacLeod wrote: Is anyone else seeing comparison problems on trunk? I was having problems testing a patch on a 4/1

Re: [PATCH][PR65823] Fix va_arg ap_copy nop detection

2015-04-22 Thread Richard Biener
On Wed, Apr 22, 2015 at 3:38 PM, Tom de Vries wrote: > On 22-04-15 10:06, Richard Biener wrote: >> >> On Wed, Apr 22, 2015 at 9:41 AM, Tom de Vries >> wrote: >>> >>> Hi, >>> >>> this patch fixes PR65823. >>> > > > >>> >>> The patches fixes the problem by using operand_equal_p to do the equality

[PATCH] Quiet down -Wlogical-op a bit (PR c/61534)

2015-04-22 Thread Marek Polacek
This patch stifles -Wlogical-op a bit: don't warn if either operand comes from a macro expansion. As the comment says, it doesn't fix the bug completely, but it's a simple improvement. I did this by introducing a new macro. Bootstrapped/regtested on x86_64-linux, ok for trunk? (Bootstrap with -W

[PATCH] Dev tree housekeeping

2015-04-22 Thread Richard Biener
This pushes a bunch of changes from my dev tree to trunk. Bootstrapped on x86_64-unknown-linux-gnu, testing in progress. Richard. 2015-04-22 Richard Biener * cfgexpand.c (expand_gimple_stmt_1): Use ops.code. * cfgloop.c (verify_loop_structure): Verify the root loop node.

Re: [PATCH 1/13] libitm fixes for musl support

2015-04-22 Thread Kyrill Tkachov
On 20/04/15 21:41, Jeff Law wrote: On 04/20/2015 12:51 PM, Szabolcs Nagy wrote: This are minor correctness fixes required for musl. (fcntl.h is the standard header and always available on Linux, sys/fcntl.h is just a legacy alias, so use the standard one.) libitm/Changelog: 2015-04-16 Grego

Re: [PATCH 2/13] musl libc config

2015-04-22 Thread Kyrill Tkachov
On 22/04/15 14:16, Jeff Law wrote: On 04/20/2015 12:52 PM, Szabolcs Nagy wrote: Add musl libc support to gcc and the command line option -mmusl following other libc support code. Note that -m cannot be entirely correct: there are build time decisions based on the default libc. gcc/Changelog:

Re: [PATCH 11/13] unwind fix for musl

2015-04-22 Thread Kyrill Tkachov
On 22/04/15 14:20, Jeff Law wrote: On 04/20/2015 12:58 PM, Szabolcs Nagy wrote: dl_iterate_phdr depends on USE_PT_GNU_EH_FRAME. I think USE_PT_GNU_EH_FRAME could be enabled more generally (whenever libc provides dl_iterate_phdr), but I only made a conservative change. libgcc/Changelog: 2015

Re: [PATCH 12/13] libstdc++, libgfortran gthr workaround for musl

2015-04-22 Thread Kyrill Tkachov
On 22/04/15 14:17, Jeff Law wrote: On 04/20/2015 12:59 PM, Szabolcs Nagy wrote: libgcc/gthr-posix.h uses weak reference logic to determine if libpthread is linked into the application or not. This is broken unless there is special workaround with libc internal knowledge and even then static li

Re: [PATCH 10/13] fixincludes

2015-04-22 Thread Kyrill Tkachov
On 20/04/15 21:38, Jeff Law wrote: On 04/20/2015 12:58 PM, Szabolcs Nagy wrote: No fixincludes are needed for musl. fixincludes/Changelog: 2015-04-16 Gregor Richards * mkfixinc.sh: Add *-musl* with no fixes. OK. jeff I've committed this on Szabolcs' behalf with r222327. Kyril

Re: [PATCH][AArch64] Implement -m{cpu,tune,arch}=native using only /proc/cpuinfo

2015-04-22 Thread Kyrill Tkachov
On 22/04/15 12:46, Kyrill Tkachov wrote: [Sorry for resending twice. My mail client glitched] On 20/04/15 16:47, Kyrill Tkachov wrote: Hi all, This is an attempt to add native CPU detection to AArch64 GNU/Linux targets. Similar to other ports we use SPEC rewriting to rewrite -m{cpu,tune,arch}

Re: [WIP] OpenMP 4 NVPTX support

2015-04-22 Thread Bernd Schmidt
On 04/21/2015 05:58 PM, Jakub Jelinek wrote: suggests that while it is nice that when building nvptx accel compiler we build libgcc.a, libc.a, libm.a, libgfortran.a (and in the future hopefully libgomp.a), nothing attempts to link those in :(. I have that fixed; I expect I'll get around to po

[PATCH] Fix up tm_clone_hasher

2015-04-22 Thread Marek Polacek
handle_cache_entry in tm_clone_hasher looks wrong: the condition if (e != HTAB_EMPTY_ENTRY || e != HTAB_DELETED_ENTRY) is always true. While it could be fixed by just changing || into &&, I decided to follow suit and do what we do in handle_cache_entry's elsewhere in the codebase. I've fixed a fo

Re: [PATCH 2/13] musl libc config

2015-04-22 Thread Ilya Verbin
On Wed, Apr 22, 2015 at 15:34:51 +0100, Kyrill Tkachov wrote: > > On 22/04/15 14:16, Jeff Law wrote: > >On 04/20/2015 12:52 PM, Szabolcs Nagy wrote: > >>Add musl libc support to gcc and the command line option -mmusl following > >>other > >>libc support code. > >> > >>Note that -m cannot be entir

RE: [PATCH 2/13] musl libc config

2015-04-22 Thread Kyrill Tkachov
On 22/04/15 16:26, Ilya Verbin wrote: > On Wed, Apr 22, 2015 at 15:34:51 +0100, Kyrill Tkachov wrote: >> On 22/04/15 14:16, Jeff Law wrote: >>> On 04/20/2015 12:52 PM, Szabolcs Nagy wrote: Add musl libc support to gcc and the command line option -mmusl following other libc support code.

Re: [PATCH 2/13] musl libc config

2015-04-22 Thread Szabolcs Nagy
On 22/04/15 16:36, Kyrylo Tkachov wrote: > On 22/04/15 16:26, Ilya Verbin wrote: >> On Wed, Apr 22, 2015 at 15:34:51 +0100, Kyrill Tkachov wrote: >>> On 22/04/15 14:16, Jeff Law wrote: On 04/20/2015 12:52 PM, Szabolcs Nagy wrote: > Add musl libc support to gcc and the command line option

RE: [PATCH v3][MIPS] fix CRT_CALL_STATIC_FUNCTION macro

2015-04-22 Thread Petar Jovanovic
-Original Message- From: Maciej W. Rozycki [mailto:ma...@linux-mips.org] Sent: Tuesday, April 21, 2015 8:52 PM To: Petar Jovanovic Cc: gcc-patches@gcc.gnu.org; catherine_mo...@mentor.com; matthew.fort...@imgtec.com Subject: Re: [PATCH v3][MIPS] fix CRT_CALL_STATIC_FUNCTION macro > I think

[PATCH][AArch64] Properly cost FABD pattern

2015-04-22 Thread Kyrill Tkachov
Hi all, In rtx costs we do not handle the FP abs (minus (a b)) case which maps down to a FABD instruction. This patch fixes that. FABD behaves similarly to the FADD class of instructions unlike simple FABS which is closer to FNEG. Tested aarch64-none-elf. Ok for trunk? Thanks, Kyrill 2015-04-22

[PATCH][ARM] Do not lower cost of setting core reg to constant. It doesn't have any effect

2015-04-22 Thread Kyrill Tkachov
Hi all, This hunk that slightly reduces the cost of immediate moves doesn't actually have any effect. In the whole of SPEC2006 it didn't make a difference. In any case, I'd like to move to a point where we use COSTS_N_INSNS units for our costs and not increment decrement them by one. This pat

[PATCH] PR target/65846: Optimize data access in PIE with copy reloc

2015-04-22 Thread H.J. Lu
Normally, with PIE, GCC accesses globals that are extern to the module using GOT. This is two instructions, one to get the address of the global from GOT and the other to get the value. Examples: --- extern int a_glob; int main () { return a_glob; } --- With PIE, the generated code accesses g

[PATCH 0/14][ARM/AArch64] __FP16 support, vectors, intrinsics, testsuite

2015-04-22 Thread Alan Lawrence
This patch series adds support for ARM Neon float16x4_t and float16x8_t vector types and intrinsics, and the __fp16 type, on both ARM and AArch64, and extends the tests in Christophe Lyon's advsimd-intrinsics testsuite to cover these. (I chose to extend the existing tests rather than add new one

Re: [PATCH 1/2][ARM] PR/63870: Add qualifier to check lane bounds in expand

2015-04-22 Thread Alan Lawrence
Ping (https://gcc.gnu.org/ml/gcc-patches/2015-01/msg01422.html). These are required for float16 patches posted at https://gcc.gnu.org/ml/gcc-patches/2015-04/msg01332.html . Bootstrapped + check-gcc on arm-none-linux-gnueabihf. Alan Lawrence wrote: This is based loosely upon svn r217440, "[AAr

Re: [PATCH 2/2][ARM] PR/63870: Add a __builtin_lane_check

2015-04-22 Thread Alan Lawrence
Ping (https://gcc.gnu.org/ml/gcc-patches/2015-01/msg01436.html). These are required for float16 patches posted at https://gcc.gnu.org/ml/gcc-patches/2015-04/msg01332.html Bootstrapped + check-gcc on arm-none-linux-gnueabihf. Alan Lawrence wrote: This parallels the present form of __builtin_aa

[PATCH 2/14][ARM]Add float16x8_t type

2015-04-22 Thread Alan Lawrence
Identical to https://gcc.gnu.org/ml/gcc-patches/2015-01/msg01438.html . Bootstrapped on arm-none-linux-gnueabihf. commit bc582bd6a0ed7c7c91fc834603fc573ed745b1a7 Author: Alan Lawrence Date: Mon Dec 8 18:40:24 2014 + Add float16x8_t + V8HFmode support (regardless of -mfp16-format) dif

[PATCH 1/14][ARM] Add float16x4_t intrinsics

2015-04-22 Thread Alan Lawrence
This is a respin of https://gcc.gnu.org/ml/gcc-patches/2015-01/msg01437.html , but fixes a wrong 'lane index out of bounds' error on vget_lane_f16 and vset_lane_f16, and drops vdup_n_f16 and vdup_lane_f16, as these are not in the ACLE spec. As previously, these use GCC vector extensions to maxim

[PATCH 3/14][ARM] Add float16x8_t intrinsics

2015-04-22 Thread Alan Lawrence
This is a respin of https://gcc.gnu.org/ml/gcc-patches/2015-01/msg01439.html , again fixing a wrong 'lane index out of bounds' error for vgetq_lane_f16 and vsetq_lane-f16 at -O0, and dropping vdupq_n_f16 and vdupq_lane_f16 as these are not in the ACLE spec. The vld1, vldN, vldN_lane and corres

[PATCH 4/14][ARM] Remaining float16 intrinsics: vld..., vst..., vget_low|high, vcombine

2015-04-22 Thread Alan Lawrence
This is a respin of https://gcc.gnu.org/ml/gcc-patches/2015-01/msg01440.html ; changes are to add in several missing vst... intrinsics, and fix a missing iterator V_uf_sclr used in vec_extract. These intrinsics are all made from patterns in neon.md, and are all tied together by iterators - I'v

[PATCH 5/14][AArch64] Add basic fp16 support

2015-04-22 Thread Alan Lawrence
This adds basic support for moving __fp16 values around, passing and returning, and operating on them by promoting to 32-bit floats. Also a few scalar testcases. Note I've not got an fmov (immediate) variant, because there is no 'fmov h, ...' - the only way to load a 16-bit immediate is to rein

[PATCH 5/14][AArch64] Add basic fp16 support

2015-04-22 Thread Alan Lawrence
[Resending with correct in-reply-to header] This adds basic support for moving __fp16 values around, passing and returning, and operating on them by promoting to 32-bit floats. Also a few scalar testcases. Note I've not got an fmov (immediate) variant, because there is no 'fmov h, ...' - the

[PATCH 6/14][AArch64] Add support for float16x{4,8}_t vectors/builtins

2015-04-22 Thread Alan Lawrence
This adds some basic intrinsics - vget_lane, vset_lane, vld1_lane, vld1, vst1 - for float16 types, and the necessary support in the builtin generator, basic patterns for moving values around, etc. Other intrinsics will follow in later patches. I've extended the existing testcases in aarch64/,

[PATCH 7/14][AArch64] vld{2,3,4}{,_lane,_dup},vcombine,vcreate

2015-04-22 Thread Alan Lawrence
gcc/ChangeLog: * config/aarch64/aarch64.c (aarch64_split_simd_combine): Add V4HFmode. * config/aarch64/aarch64-builtins.c (VAR13, VAR14): New. (aarch64_scalar_builtin_types, aarch64_init_simd_builtin_scalar_types): Add __builtin_aarch64_simd_hf. * config/aa

[PATCH 8/14][AArch64]Add vreinterpret, float_truncate_lo/hi, vget_low/high

2015-04-22 Thread Alan Lawrence
gcc/ChangeLog: * config/aarch64/aarch64-simd.md (aarch64_float_truncate_lo_v2sf): Reparameterize to... (aarch64_float_truncate_lo_): ...this, for both V2SF and V4HF. (aarch64_float_truncate_hi_v4sf): Reparameterize to... (aarch64_float_truncate_hi_): ...thi

[PATCH 9/14][AArch64] vld1(q?)_dup, missing vreinterpretq intrinsics

2015-04-22 Thread Alan Lawrence
gcc/ChangeLog: * config/aarch64/arm_neon.h (vreinterpretq_p8_f16, vreinterpretq_p16_f16, vreinterpretq_f32_f16, vreinterpretq_f64_f16, vreinterpretq_s64_f16, vreinterpretq_s8_f16, vreinterpretq_s16_f16, vreinterpretq_s32_f16, vreinterpretq_u8_f16, vreinterpretq_u16

[PATCH 10/14][AArch64] Add vcvt(_high)?_f32_f16 intrinsics

2015-04-22 Thread Alan Lawrence
This adds the two remaining widening intrinsics, first adding patterns in aarch64-simd.md, then entries in aarch64-simd-builtins.def, and finally intrinsics in arm_neon.h . Note this changes the vector indices present in the RTL on bigendian for float vec_unpacks, to be the same as for integer

[PATCH 11/14][fold-const.c] Fix bigendian HFmode in native_interpret_real

2015-04-22 Thread Alan Lawrence
native_interpret_real in fold_const.c has an assumption that floats are at least 32-bits (on bigendian targets with UNITS_PER_WORD >= 4). This patch relaxes that assumption (allowing e.g. 16-bit HFmode values). On aarch64_be-none-elf, this fixes the float16x4_t variant of gcc.target/aarch64/ad

[PATCH 12/14][ARM/AArch64 Testsuite] Update advsimd-intrinsics tests to add float16 vectors

2015-04-22 Thread Alan Lawrence
This is a fairly straightforward addition of a new type: I've added it in on equal status to the other types, because the various vector-load/store/element-manipulating intrinsics, are *not* conditional on HW support. (They just involve moving 16-bit chunks around, just like s16/u16/p16). Thus

[PATCH 13/14][ARM/AArch64 testsuite] Use gcc-dg-runtest in advsimd-intrinsics.exp

2015-04-22 Thread Alan Lawrence
In the first revision of Christophe Lyon's advsimd-intrinsics tests, https://gcc.gnu.org/ml/gcc-patches/2014-06/msg00532.html , both gcc-dg-runtest (to assemble only) and c-torture-execute were used. In review the gcc-dg-runtest part was then dropped, and execution tests continued using c-tortur

[PATCH 14/14][ARM/AArch64 testsuite] Test float16_t vcvt_* intrinsics

2015-04-22 Thread Alan Lawrence
This adds a test of vcvt_f32_f16 and vcvt_f16_f32, also vcvt_high_f32_f16 and vcvt_high_f16_f32. On ARM, we pass additional option -mfpu=neon-fp16 to the compiler (possible following patch 2/3). The compiler is already receiving an option such as -mfpu=neon or -mfpu=crypto-neon-fp-armv8, but p

Re: [PATCH 7/14][AArch64] vld{2,3,4}{,_lane,_dup},vcombine,vcreate

2015-04-22 Thread Alan Lawrence
Alan Lawrence wrote: gcc/ChangeLog: * config/aarch64/aarch64.c (aarch64_split_simd_combine): Add V4HFmode. * config/aarch64/aarch64-builtins.c (VAR13, VAR14): New. (aarch64_scalar_builtin_types, aarch64_init_simd_builtin_scalar_types): Add __builtin_aarch64_simd_h

Re: [PATCH 00/12] Reduce conditional compilation

2015-04-22 Thread David Malcolm
On Tue, 2015-04-21 at 11:29 -0400, Trevor Saunders wrote: > On Tue, Apr 21, 2015 at 07:57:19AM -0600, Jeff Law wrote: > > On 04/21/2015 07:24 AM, tbsaunde+...@tbsaunde.org wrote: > > >From: Trevor Saunders > > > > > >Hi, > > > > > >This is a first round of patches to reduce the amount of code with

Re: [PATCH] Makefile.tpl: Remove surplus whitespace

2015-04-22 Thread Bernhard Reutner-Fischer
On 9 April 2015 at 13:20, Bernhard Reutner-Fischer wrote: > essentially s/[[:space:]]*[[:space:]];/;/g > > ChangeLog (attn: to src, IIRC no write-access, ask someone to commit) > > Ok for trunk now? Jeff OKed this, applied as r222334. Please holler if i broke something.. thanks, > > 2015-04-01

[C/C++ PATCH] Implement -Wshift-negative-value (PR c/65179)

2015-04-22 Thread Marek Polacek
Currently, we warn if the right operand of a shift expression is negative, or greater than or equal to the length in bits of the promoted left operand. But we don't warn when we see a left shift of a negative value. That is undefined behavior since C99 and I believe C++11, so this patch implement

  1   2   >