[09/12] Remove all but one use of default_hashmap_traits

2015-06-23 Thread Richard Sandiford
After the previous patches in the series, there are three remaining hash traits that use the key to represent empty and deleted entries. This patch makes them use simple_hashmap_traits. gcc/ * ipa-icf.h (symbol_compare_hash): New class. (symbol_compare_hashmap_traits): Use it.

Re: [gomp4.1] Add new versions of GOMP_target{,_data,_update} and GOMP_target_enter_exit_data

2015-06-23 Thread Jakub Jelinek
On Tue, Jun 23, 2015 at 05:54:48PM +0300, Ilya Verbin wrote: On Tue, Jun 23, 2015 at 13:51:39 +0200, Jakub Jelinek wrote: + /* Set dd on target to 0 for the further check. */ + #pragma omp target map(always to: dd) + { dd; } This reminds me that: if

[committed] Use abort in parloops-exit-first-loop-alt-{3,4}.c

2015-06-23 Thread Tom de Vries
Hi, committed attached patch as trivial. Thanks, - Tom Use abort in parloops-exit-first-loop-alt-{3,4}.c 2015-06-23 Tom de Vries t...@codesourcery.com * testsuite/libgomp.c/parloops-exit-first-loop-alt-3.c (main): Use abort. * testsuite/libgomp.c/parloops-exit-first-loop-alt-4.c (main):

[03/12] Move decl hasher to header file

2015-06-23 Thread Richard Sandiford
Like the previous patch, but for decl hashers. There's only one copy of this so far, but the idea seems general. gcc/ * tree-hash-traits.h (tree_decl_hash): New class. * tree-ssa-strlen.c: Include tree-hash-traits.h. (stridxlist_hash_traits): Use tree_decl_hash. Index:

[06/12] Consolidate string hashers

2015-06-23 Thread Richard Sandiford
This patch replaces various string hashers with a single copy in hash-traits.h. gcc/ * hash-traits.h (string_hash, nofree_string_hash): New classes. * genmatch.c (capture_id_map_hasher): Use nofree_string_hash. * passes.c (pass_registry_hasher): Likewise. *

[05/12] Move TREE_HASH hasher to header file

2015-06-23 Thread Richard Sandiford
One more tree hasher, this time based on TREE_HASH. gcc/ * tree-hash-traits.h (tree_hash): New class. * except.c: Include tree-hash-traits.h. (tree_hash_traits): Use tree_hash. Index: gcc/tree-hash-traits.h

Re: [PATCH] c/66516 - missing diagnostic on taking the address of a builtin function

2015-06-23 Thread Martin Sebor
On 06/23/2015 04:29 AM, Jakub Jelinek wrote: On Tue, Jun 23, 2015 at 12:18:30PM +0200, Marek Polacek wrote: Is it intended that programs be able to take the address of the builtins that correspond to libc functions and make calls to the underlying libc functions via such pointers? (If so, the

Remove redundant AND from count reduction loop

2015-06-23 Thread Richard Sandiford
We vectorise: int f (int *a, int n) { int count = 0; for (int i = 0; i n; ++i) if (a[i] 255) count += 1; return count; } using an add reduction of a VEC_COND_EXPR COND, {1, 1, ...}, {0, 0, ...}. This leads to the main loop having an AND with a loop invariant {1, 1, ...}. E.g.

Re: [Patch SRA] Fix PR66119 by calling get_move_ratio in SRA

2015-06-23 Thread James Greenhalgh
On Tue, Jun 23, 2015 at 09:52:01AM +0100, Jakub Jelinek wrote: On Tue, Jun 23, 2015 at 09:18:52AM +0100, James Greenhalgh wrote: This patch fixes the issue by always calling get_move_ratio in the SRA code, ensuring that an up-to-date value is used. Unfortunately, this means we have to

[10/12] Add helper class for valued-based empty and deleted slots

2015-06-23 Thread Richard Sandiford
part_traits in cfgexpand.c needs to use the value rather than the key to represent empty and deleted slots. What it's doing is pretty generic, so this patch adds a helper class to hash-map-traits.h. gcc/ * hash-map-traits.h (unbounded_hashmap_traits): New class.

Re: [gomp4.1] Add new versions of GOMP_target{,_data,_update} and GOMP_target_enter_exit_data

2015-06-23 Thread Ilya Verbin
On Tue, Jun 23, 2015 at 13:51:39 +0200, Jakub Jelinek wrote: + /* Set dd on target to 0 for the further check. */ + #pragma omp target map(always to: dd) + { dd; } This reminds me that: if (ctx-region_type == ORT_TARGET !(n-value GOVD_SEEN)) remove =

[02/12] Move tree operand hashers to a new header file

2015-06-23 Thread Richard Sandiford
There were three tree operand hashers, so move them to their own header file. The typedefs in this and subsequent patches are temporary and get removed in patch 12. gcc/ * tree-hash-traits.h: New file. (tree_operand_hash): New class. * sanopt.c: Include

[04/12] Move ssa_name hasher to header file

2015-06-23 Thread Richard Sandiford
Another tree hasher, this time for SSA names. Again there's only one copy at the moment, but the idea seems general. gcc/ * tree-hash-traits.h (tree_ssa_name_hasher): New class. * sese.c: Include tree-hash-traits.h. (rename_map_hasher): Use tree_ssa_name_hasher. Index:

[12/12] Simplify uses of hash_map

2015-06-23 Thread Richard Sandiford
At this point all hash_map traits know what kind of key they're dealing with, so we can make that a traits typedef, like it is for hash_table traits. Then, if we make the default hash traits for T be T, we can use hash_table-style traits as the first template parameter to hash_map, without the

[PATCH] Make muser-mode the default for LEON3

2015-06-23 Thread Daniel Cederman
The muser-mode flag causes the CASA instruction for LEON3 to use the user mode ASI. This is the correct behavior for almost all LEON3 targets. For this reason it makes sense to make user mode the default. This patch adds a flag for supervisor mode that can be used on the very few LEON3 targets

[PATCH 1/2] Add mask to specify which LEON3 targets support CASA

2015-06-23 Thread Daniel Cederman
Not all LEON3 support the CASA instruction. This patch provides a mask that can be used to specify which LEON3 targets that support CASA. gcc/ChangeLog: 2015-06-22 Daniel Cederman ceder...@gaisler.com * config/sparc/sparc.c (sparc_option_override): Mark CPU targets leon3 and

C++ PATCH for c++/66501 (wrong code with array move assignment)

2015-06-23 Thread Jason Merrill
build_vec_init was assuming that if a class has a trivial copy assignment, then an array assignment is trivial. But overload resolution might not choose the copy assignment operator. So this patch changes build_vec_init to check for any non-trivial assignment operator. Tested

[01/12] Add hash_map traits that use existing hash_table-like traits

2015-06-23 Thread Richard Sandiford
This patch defines a class that converts hash_table-style traits into hash_map traits. It can be used as the default traits for all hash_maps that don't specify their own traits (i.e. this patch does work on its own). By the end of the series this class replaces default_hashmap_traits. gcc/

[07/12] Use new string hasher for MIPS

2015-06-23 Thread Richard Sandiford
Use the string hasher from patch 6 for MIPS. I split this out because local_alias_traits doesn't actually need to use SYMBOL_REF rtxes as the map keys, since the only data used is the symbol name. gcc/ * config/mips/mips.c (mips16_flip_traits): Use it. (local_alias_traits,

[08/12] Add common traits for integer hash keys

2015-06-23 Thread Richard Sandiford
Several places define hash traits for integers, using particular integer values as empty and deleted markers. This patch defines them in terms of a single int_hash class. I also needed to extend gengtype to accept + in template arguments. gcc/ * gengtype-parse.c

Re: [PATCH] Add CFI entries for ARM Linux idiv0 / ldiv0

2015-06-23 Thread Ramana Radhakrishnan
On 16/06/15 22:25, James Lemke wrote: A divide by zero exception was not giving a proper traceback for LINUX ARM_EABI. The attached patch fixes the problem on trunk (and several local branches). Tested on gcc-trunk for arm-none-linux-gnueabi. OK to commit? 2015-06-16 James Lemke

[00/12] Share hash traits between hash_table and hash_map

2015-06-23 Thread Richard Sandiford
Following on from: https://gcc.gnu.org/ml/gcc-patches/2015-06/msg01066.html this series unifies the key hashing traits for hash_maps in all but one case (which needs to use the values rather than keys to represent deleted and empty entries). It also consolidates the various tree, integer and

[gomp4.1] Taskloop C++ random access iterator support

2015-06-23 Thread Jakub Jelinek
Hi! I've committed following patch to add support for C++ random access iterators in taskloop constructs. 2015-06-23 Jakub Jelinek ja...@redhat.com * tree.h (OMP_CLAUSE_PRIVATE_TASKLOOP_IV, OMP_CLAUSE_LASTPRIVATE_TASKLOOP_IV): Define. * gimplify.c (gimplify_omp_for):

[11/12] Remove default_hashmap_traits

2015-06-23 Thread Richard Sandiford
The previous patches removed all uses of default_hashmap_traits, so this patch deletes the definition. gcc/ * hash-map-traits.h (default_hashmap_traits): Delete. Index: gcc/hash-map-traits.h === --- gcc/hash-map-traits.h

Re: [patch] Delete temporary response file

2015-06-23 Thread Jeff Law
On 06/22/2015 11:37 AM, Eric Botcazou wrote: Hi, when you pass a response file at link time and you use the GNU linker, then collect2 creates another, temporary response file and passes it to the linker. But it fails to delete the file after it is done. This can easily be seen with the

Re: [PATCH] Make muser-mode the default for LEON3

2015-06-23 Thread Sebastian Huber
Instead of introducing a new option which may conflict with an existing one, is it not possible to simply use -mno-user-mode? On 23/06/15 14:22, Daniel Cederman wrote: The muser-mode flag causes the CASA instruction for LEON3 to use the user mode ASI. This is the correct behavior for almost

Re: [AArch64] Implement -fpic for -mcmodel=small

2015-06-23 Thread Jiong Wang
Marcus Shawcroft writes: On 20 May 2015 at 11:21, Jiong Wang jiong.w...@arm.com wrote: gcc/ * config/aarch64/aarch64.md: (ldr_got_small_mode): Support new GOT relocation modifiers. (ldr_got_small_sidi): Ditto. * config/aarch64/iterators.md (got_modifier): New mode iterator. *

C++ PATCH for c++/65879 (error on member function of nested class of anonymous class)

2015-06-23 Thread Jason Merrill
It doesn't make sense to complain about a function using its own enclosing class. There were two problems here: 1) The function should have been marked as internal because its class is internal. 2) We shouldn't bother looking at 'this' for no-linkage types. Tested x86_64-pc-linux-gnu,

Re: [PATCH] Make muser-mode the default for LEON3

2015-06-23 Thread Daniel Cederman
On 2015-06-23 14:34, Jakub Jelinek wrote: On Tue, Jun 23, 2015 at 02:22:34PM +0200, Daniel Cederman wrote: The muser-mode flag causes the CASA instruction for LEON3 to use the user mode ASI. This is the correct behavior for almost all LEON3 targets. For this reason it makes sense to make user

[c-family PATCH] Fix for -Wlogical-op

2015-06-23 Thread Marek Polacek
While looking at something else I noticed that we're using == for INTEGER_CSTs comparison. That isn't going to work well, so use tree_int_cst_equal instead. Because of that we weren't diagnosing the following test. Bootstrapped/regtested on x86_64-linux, ok for trunk? 2015-06-23 Marek Polacek

Re: [AArch64] Implement -fpic for -mcmodel=small

2015-06-23 Thread Marcus Shawcroft
On 23 June 2015 at 14:02, Jiong Wang jiong.w...@arm.com wrote: Marcus Shawcroft writes: On 20 May 2015 at 11:21, Jiong Wang jiong.w...@arm.com wrote: gcc/ * config/aarch64/aarch64.md: (ldr_got_small_mode): Support new GOT relocation modifiers. (ldr_got_small_sidi): Ditto. *

Re: RFA: Add support for -fstack-usage to various ports

2015-06-23 Thread Jeff Law
On 06/23/2015 06:56 AM, Nick Clifton wrote: Hi Guys, The patch below adds support for the -fstack-usage option to the BFIN, FT32, H8300, IQ2000 and M32C ports. It also adjusts the expected output in the gcc.dg/stack-usage-1.c test for the V850 and MN10300 to match the actual

Re: [PATCH] Make muser-mode the default for LEON3

2015-06-23 Thread Daniel Cederman
On 2015-06-23 14:58, Jakub Jelinek wrote: On Tue, Jun 23, 2015 at 02:48:45PM +0200, Daniel Cederman wrote: How does one check if the bit has been explicitly set? It was not obvious to if (TARGET_USER_MODE_P (target_flags_explicit)) me, which is why I took a similar approach to a patch I

Re: [PATCH] Make muser-mode the default for LEON3

2015-06-23 Thread Jakub Jelinek
On Tue, Jun 23, 2015 at 02:22:34PM +0200, Daniel Cederman wrote: The muser-mode flag causes the CASA instruction for LEON3 to use the user mode ASI. This is the correct behavior for almost all LEON3 targets. For this reason it makes sense to make user mode the default. This patch adds a flag

Re: [c-family PATCH] Fix for -Wlogical-op

2015-06-23 Thread Jeff Law
On 06/23/2015 07:12 AM, Marek Polacek wrote: While looking at something else I noticed that we're using == for INTEGER_CSTs comparison. That isn't going to work well, so use tree_int_cst_equal instead. Because of that we weren't diagnosing the following test. Bootstrapped/regtested on

RFA: Add support for -fstack-usage to various ports

2015-06-23 Thread Nick Clifton
Hi Guys, The patch below adds support for the -fstack-usage option to the BFIN, FT32, H8300, IQ2000 and M32C ports. It also adjusts the expected output in the gcc.dg/stack-usage-1.c test for the V850 and MN10300 to match the actual results generated by these toolchains. Tested with no

[PATCH] Make muser-mode the default for LEON3

2015-06-23 Thread Daniel Cederman
The muser-mode flag causes the CASA instruction for LEON3 to use the user mode ASI. This is the correct behavior for almost all LEON3 targets. For this reason it makes sense to make user mode the default. gcc/ChangeLog: 2015-06-23 Daniel Cederman ceder...@gaisler.com *

[PATCH 2/2] Add leon3r0 and leon3r0v7 CPU targets

2015-06-23 Thread Daniel Cederman
Early variants of LEON3, revision 0, do not support the CASA instruction. This patch adds two new targets, leon3r0 and leon3r0v7, that are equivalent to leon3 and leon3v7, except that they do not support CASA. gcc/ChangeLog: 2015-06-22 Daniel Cederman ceder...@gaisler.com *

[PATCH] Use leon3 target for native LEON on Linux

2015-06-23 Thread Daniel Cederman
Linux requires LEON version 3 or above with CASA support. gcc/ChangeLog: 2015-06-23 Daniel Cederman ceder...@gaisler.com * config/sparc/driver-sparc.c: map /proc/cpuinfo with CPU LEON to leon3 --- gcc/config/sparc/driver-sparc.c | 1 + 1 file changed, 1 insertion(+) diff

Re: [PATCH][ARM] PR/65711: Don't pass '-dynamic-linker' when '-shared' is used

2015-06-23 Thread Ludovic Courtès
Jakub Jelinek ja...@redhat.com skribis: On Tue, Jun 23, 2015 at 11:30:25AM +0200, Ludovic Courtès wrote: Ramana Radhakrishnan ramana@googlemail.com skribis: On Thu, Apr 23, 2015 at 9:29 AM, Ludovic Courtès l...@gnu.org wrote: As discussed at

[gomp4] Additional testing for deviceptr clause.

2015-06-23 Thread James Norris
Hi! The following patch adds additional testing of the deviceptr clause. Patch applied to gomp-4_0-branch. Thanks! Jim diff --git a/libgomp/testsuite/libgomp.oacc-c-c++-common/deviceptr-1.c b/libgomp/testsuite/libgomp.oacc-c-c++-common/deviceptr-1.c index e271a37..e62c315 100644 ---

Re: [PATCH] Make muser-mode the default for LEON3

2015-06-23 Thread Jakub Jelinek
On Tue, Jun 23, 2015 at 02:48:45PM +0200, Daniel Cederman wrote: How does one check if the bit has been explicitly set? It was not obvious to if (TARGET_USER_MODE_P (target_flags_explicit)) me, which is why I took a similar approach to a patch I found for another CPU target. If it is possible

Re: [PATCH] Expand PIC calls without PLT with -fno-plt

2015-06-23 Thread Jeff Law
On 06/23/2015 02:29 AM, Ramana Radhakrishnan wrote: If you try disabling the REG_EQUAL note generation [*], you'll probably find a performance regression on arm32 (and probably on aarch64 as well? we only IMHO disabling the REG_EQUAL note generation is the wrong way to go about this. Agreed.

[nvptx] add select

2015-06-23 Thread Nathan Sidwell
I've committed this PTX patch to add support for the selp instruction. It's pretty much a direct implementation of 'r = a ? b : c'. This is sufficient for combine(?) to generate selp instructions such as: selp.u32%r22, %r25, %r26, %r28; selp.u32%r22, %r25, 4, %r27;

Re: RFA: FT32: Fix building gcc.

2015-06-23 Thread Jeff Law
On 06/23/2015 06:10 AM, Nicholas Clifton wrote: Hi Guys, It seems that the FT32 port of GCC does not have a maintainer at the moment. Nevertheless I have a patch to fix a couple of build time problems compiling gcc for the FT32. Is this OK to apply ? Cheers Nick gcc/ChangeLog

[PATCH] Fix PR66636

2015-06-23 Thread Richard Biener
The following fixes an ICE with 188.ammp and AVX2. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. Richard. 2015-06-23 Richard Biener rguent...@suse.de PR tree-optimization/66636 * tree-vect-stmts.c (vectorizable_store): Properly compute the

Fwd: [PATCH] Add CFI entries for ARM Linux idiv0 / ldiv0

2015-06-23 Thread James Lemke
Ping.. Forwarded Message Subject: [PATCH] Add CFI entries for ARM Linux idiv0 / ldiv0 Date: Tue, 16 Jun 2015 17:25:49 -0400 From: James Lemke jwle...@codesourcery.com To: gcc-patches@gcc.gnu.org A divide by zero exception was not giving a proper traceback for LINUX ARM_EABI.

Re: [gomp4] Generate sequential loop for OpenACC loop directive inside kernels

2015-06-23 Thread Chung-Lin Tang
On 2015/6/16 05:05 PM, Tom de Vries wrote: On 16/06/15 10:59, Chung-Lin Tang wrote: This patch adjusts omp-low.c:expand_omp_for_generic() to expand to a sequential loop form (without the OMP runtime calls), used for loop directives inside OpenACC kernels constructs. Tom mentions that this

C++ PATCH for c++/66542 (missing error with deleted dtor and static variable)

2015-06-23 Thread Jason Merrill
In expand_static_init we were diagnosing a deleted dtor if there was also no initializer, but not if there was, and nothing later on was diagnosing it either. Fixed thus. Tested x86_64-pc-linux-gnu, applying to trunk. commit 78593d02fb6af72a8f97e52cbfbbe9f49b29e9db Author: Jason Merrill

[PATCH 2/3][AArch64 nofp] Clarify docs for +nofp/-mgeneral-regs-only

2015-06-23 Thread Alan Lawrence
James Greenhalgh wrote: -Generate code which uses only the general registers. +Generate code which uses only the general registers. Equivalent to feature The ARMARM uses general-purpose registers to refer to these registers, we should match that style. s/Equivalent to feature/This is

[PATCH 1/3][AArch64 nofp] Fix ICEs with +nofp/-mgeneral-regs-only and improve error messages

2015-06-23 Thread Alan Lawrence
James Greenhalgh wrote: Submissions on this list should be one patch per mail, it makes tracking review easier. OK here's a respin of the first, I've added a third patch after I found another route to get to an ICE. +void +aarch64_err_no_fpadvsimd (machine_mode mode, const char *msg) +{ +

Re: *Ping* patch, fortran] Warn about constant integer divisions

2015-06-23 Thread Jerry DeLisle
On 06/23/2015 01:36 AM, Janne Blomqvist wrote: On Sun, Jun 21, 2015 at 4:57 PM, Thomas Koenig tkoe...@netcologne.de wrote: *ping* https://gcc.gnu.org/ml/gcc-patches/2015-06/msg00966.html Hello world, the attached patch emits a warning for constant integer division. While correct

[PATCH] backport FreeBSD add functionality to build PIE executables

2015-06-23 Thread Andreas Tobler
Hi all, I'm going to commit this patch to 5.1 in the next days unless someone objects. The patch is in my 5.1 tree since a longer time and I regularly post results. Thanks, Andreas 2015-06-22 Andreas Tobler andre...@gcc.gnu.org Backport from mainline 2015-05-18 Andreas

Re: [PATCH] Add CFI entries for ARM Linux idiv0 / ldiv0

2015-06-23 Thread James Lemke
Tested on gcc-trunk for arm-none-linux-gnueabi. OK to commit? 2015-06-16 James Lemke jwle...@codesourcery.com libgcc/config/arm/ * lib1funcs.S (aeabi_idiv0, aeabi_ldiv0): Add CFI entries for Linux ARM_EABI. s/for Linux ARM EABI// given you handle both

Re: [PATCH][testsuite] Fix TORTURE_OPTIONS overriding

2015-06-23 Thread James Greenhalgh
On Thu, Jun 18, 2015 at 11:10:01AM +0100, Richard Biener wrote: Currently when doing make check-gcc RUNTESTFLAGS=TORTURE_OPTIONS=\\\{ -O3 } { -O2 }\\\ dg-torture.exp you get -O3 and -O2 but also the two LTO torture option combinations. That's undesired (those are the most expensive

[PATCH 3/3][AArch64 nofp] Fix another ICE with +nofp/-mgeneral-regs-only

2015-06-23 Thread Alan Lawrence
This fixes another ICE, obtained with the attached testcase - yes, there was a way to get hold of a float, without passing an argument or going through movsf/movdf! Bootstrapped + check-gcc on aarch64-none-linux-gnu. gcc/ChangeLog: * config/aarch64/aarch64.md

Re: [PATCH] [ARM] Post-indexed addressing for NEON memory access

2015-06-23 Thread Ramana Radhakrishnan
On Fri, Jun 19, 2015 at 7:04 PM, Charles Baylis charles.bay...@linaro.org wrote: On 18 June 2014 at 11:06, Ramana Radhakrishnan ramana@googlemail.com wrote: On Tue, Jun 17, 2014 at 4:03 PM, Charles Baylis charles.bay...@linaro.org wrote: Your mention of larger vector modes prompted me to

[PATCH] backport Contribute FreeBSD unwind support (x86_64 and x86)

2015-06-23 Thread Andreas Tobler
Hi all, the next one. I'm going to commit this patch to 5.1 in the next days. Also in my tree, test results posted for amd64/i386-*-freebsd11.0 Thanks, Andreas 2015-06-22 Andreas Tobler andre...@gcc.gnu.org Backport from mainline 2015-05-27 John Marino gnu...@marino.st

PING: Re: [patch] PR debug/66482: Do not ICE in gen_formal_parameter_die

2015-06-23 Thread Aldy Hernandez
On 06/12/2015 10:07 AM, Aldy Hernandez wrote: Hi. This is now a P2, as it is causing a secondary target bootstrap to fail (s390). Aldy Sigh. I must say my head is spinning with this testcase and what we do with it (-O3), even prior to the debug-early work: void f(int p) {} int g() {

Re: [PATCH] libgomp sysctl check

2015-06-23 Thread Jakub Jelinek
On Tue, Jun 23, 2015 at 07:36:22PM +0200, Andreas Tobler wrote: Hi all, this patch fixes a long standing bug in the libgomp configury. The initial patch was correct, but the commit I did was wrong. Ok for trunk? Thanks, Andreas 2015-06-23 Andreas Tobler andre...@gcc.gnu.org

[PATCH] backport libjava signal handling for FreeBSD (amd64/i386)

2015-06-23 Thread Andreas Tobler
Hi again, number three. This one is also on my list to be committed to 5.1. Results on the usual place. Thanks, Andreas 2015-06-22 Andreas Tobler andre...@gcc.gnu.org Backport from mainline 2015-05-28 Andreas Tobler andre...@gcc.gnu.org * configure.host: Add

[PATCH] libgomp sysctl check

2015-06-23 Thread Andreas Tobler
Hi all, this patch fixes a long standing bug in the libgomp configury. The initial patch was correct, but the commit I did was wrong. Ok for trunk? Thanks, Andreas 2015-06-23 Andreas Tobler andre...@gcc.gnu.org * configure.ac: Fix check for header sys/sysctl.h * configure:

Re: [PATCH] Add CFI entries for ARM Linux idiv0 / ldiv0

2015-06-23 Thread Ramana Radhakrishnan
On Tue, Jun 23, 2015 at 5:18 PM, James Lemke jwle...@codesourcery.com wrote: Tested on gcc-trunk for arm-none-linux-gnueabi. OK to commit? 2015-06-16 James Lemke jwle...@codesourcery.com libgcc/config/arm/ * lib1funcs.S (aeabi_idiv0, aeabi_ldiv0): Add CFI entries for

Re: [PATCH] Add CFI entries for ARM Linux idiv0 / ldiv0

2015-06-23 Thread James Lemke
On 06/23/2015 12:22 PM, Ramana Radhakrishnan wrote: I have no access to a non-EABI arm target to test this, not sure if there are any intree anymore (probably vxworks ?) . It looks sane on a read - just apply it and look after regressions if anything gets reported in bugzilla. Thanks Ramana.

Re: [PATCH] backport FreeBSD add functionality to build PIE executables

2015-06-23 Thread Jakub Jelinek
On Tue, Jun 23, 2015 at 07:26:09PM +0200, Andreas Tobler wrote: Hi all, I'm going to commit this patch to 5.1 in the next days unless someone objects. The patch is in my 5.1 tree since a longer time and I regularly post results. Note, Richard announced plan to do 5.2-rc2 on July, 3rd, so

Re: [PATCH] backport FreeBSD add functionality to build PIE executables

2015-06-23 Thread Andreas Tobler
On 23.06.15 19:50, Jakub Jelinek wrote: On Tue, Jun 23, 2015 at 07:26:09PM +0200, Andreas Tobler wrote: Hi all, I'm going to commit this patch to 5.1 in the next days unless someone objects. The patch is in my 5.1 tree since a longer time and I regularly post results. Note, Richard

Re: [PATCH/RFC]: unittesting v2: as a plugin (was Re: [PATCH 00/17] RFC: Addding a unit testing framework to gcc)

2015-06-23 Thread Jeff Law
On 06/17/2015 01:48 PM, David Malcolm wrote: On Thu, 2015-06-11 at 00:18 +0200, Jakub Jelinek wrote: I wrote a custom formatter for the output (class deja_gnu_printer within unittests-plugin.c) which generates lines like this on stderr: PASS: ggc_test.tree_marking PASS: ggc_test.custom_struct

Re: [PATCH 00/17] RFC: Addding a unit testing framework to gcc

2015-06-23 Thread Mike Stump
On Jun 10, 2015, at 10:16 AM, David Malcolm dmalc...@redhat.com wrote: It doesn't have an output formatter for the DejaGnu format, but I guess I could write one. The gtest standard output format is IMHO superior to DejaGnu's since it tells you start-of-test/end-of-test on separate lines, so

Re: [PATCH] libgomp sysctl check

2015-06-23 Thread Andreas Tobler
On 23.06.15 19:47, Jakub Jelinek wrote: On Tue, Jun 23, 2015 at 07:36:22PM +0200, Andreas Tobler wrote: Hi all, this patch fixes a long standing bug in the libgomp configury. The initial patch was correct, but the commit I did was wrong. Ok for trunk? Thanks, Andreas 2015-06-23 Andreas

Move target-def.h includes after main include block

2015-06-23 Thread Richard Sandiford
This moves target-def.h includes to the end of the include list, so that the file sees the same header files as the targetm initialisation itself. I've left a gap to emphasise that the placement is deliberate. This is a prerequisite for the patch I'm about to post. It probably doesn't make much

Re: [C/C++ PATCH] PR c++/66572. Fix Wlogical-op false positive

2015-06-23 Thread Marek Polacek
On Sat, Jun 20, 2015 at 03:02:06AM +0300, Mikhail Maltsev wrote: - /* We do not warn for constants because they are typical of macro - expansions that test for features. */ - if (CONSTANT_CLASS_P (op_left) || CONSTANT_CLASS_P (op_right)) + /* We do not warn for literal constants

Re: [PATCH] backport FreeBSD add functionality to build PIE executables

2015-06-23 Thread Jakub Jelinek
On Tue, Jun 23, 2015 at 07:58:48PM +0200, Andreas Tobler wrote: On 23.06.15 19:50, Jakub Jelinek wrote: On Tue, Jun 23, 2015 at 07:26:09PM +0200, Andreas Tobler wrote: Hi all, I'm going to commit this patch to 5.1 in the next days unless someone objects. The patch is in my 5.1 tree since

Add .def file for public target instructions

2015-06-23 Thread Richard Sandiford
[A fair bit later than promised, sorry...] Mikhail posted a patch to make genflags generate the default HAVE_foo and gen_foo definitions that have recently been added to defaults.h: https://gcc.gnu.org/ml/gcc-patches/2015-06/msg00723.html I agree it'd be a good idea to generate this kind of

Re: [PATCH 00/17] RFC: Addding a unit testing framework to gcc

2015-06-23 Thread Jeff Law
On 06/10/2015 11:16 AM, David Malcolm wrote: On Wed, 2015-06-10 at 17:34 +0200, Jakub Jelinek wrote: On Wed, Jun 10, 2015 at 11:24:41AM -0400, David Malcolm wrote: I picked the Google Test framework: http://code.google.com/p/googletest/ I must say I'm not very excited about using this, it

Re: [PATCH 1/3] Refactor entry point to -Wmisleading-indentation

2015-06-23 Thread Patrick Palka
On Mon, Jun 22, 2015 at 2:56 PM, Patrick Palka patr...@parcs.ath.cx wrote: On Mon, Jun 22, 2015 at 1:29 PM, Jeff Law l...@redhat.com wrote: On 06/09/2015 11:31 AM, Patrick Palka wrote: This patch refactors the entry point of -Wmisleading-indentation from: void

Re: [PATCH 03/17] Add test-cfg.c to gcc/unittests

2015-06-23 Thread Jeff Law
On 06/10/2015 09:24 AM, David Malcolm wrote: gcc/unittests/ChangeLog: * test-cfg.c: New file. --- gcc/unittests/test-cfg.c | 319 +++ 1 file changed, 319 insertions(+) create mode 100644 gcc/unittests/test-cfg.c So this one would be a

Re: pr66345.c size_t assumption bug

2015-06-23 Thread Jeff Law
On 06/08/2015 06:58 PM, DJ Delorie wrote: The testcase for pr 66345 assumes size_t is unsigned long instead of using the real type, which causes failures on some 16-bit targets. Ok? Also, I note that some tests check for __SIZE_TYPE__ as I do below, and others use it unconditionally as a

Re: New type-based pool allocator code miscompiled due to aliasing issue?

2015-06-23 Thread Pat Haugen
On 06/18/2015 06:10 AM, Richard Biener wrote: You are right that we should call ::new just for classes that have m_ignore_type_size == false. I've come up with following patch, that I tested slightly: diff --git a/gcc/alloc-pool.h b/gcc/alloc-pool.h index 1785df5..7da5f7a 100644 ---

[committed] Remove bogus target-def.h includes

2015-06-23 Thread Richard Sandiford
target-def.h should only be included by files that want to initialise targetm. This patch removes unnecessary includes for other files. Bootstrapped regression-tested on x86_64-linux-gnu. Also tested with config-list.mk. Applied as obvious. Thanks, Richard gcc/c-family/ *

Re: [PATCH 02/17] Add test-bitmap.c to gcc/unittests

2015-06-23 Thread Jeff Law
On 06/10/2015 09:24 AM, David Malcolm wrote: gcc/unittests/ChangeLog: * test-bitmap.c: New file. --- gcc/unittests/test-bitmap.c | 117 1 file changed, 117 insertions(+) create mode 100644 gcc/unittests/test-bitmap.c diff --git

Re: Do not take address of empty string front

2015-06-23 Thread François Dumont
On 22/06/2015 17:10, Jonathan Wakely wrote: On 20/06/15 12:59 +0100, Jonathan Wakely wrote: On 20/06/15 12:03 +0200, François Dumont wrote: Hi 2 experimental tests are failing in debug mode because __do_str_codecvt is sometimes taking address of string front() and back() even if empty. It

Re: [PATCH][RFC] Add FRE in pass_vectorize

2015-06-23 Thread Jeff Law
On 06/10/2015 08:02 AM, Richard Biener wrote: The following patch adds FRE after vectorization which is needed for IVOPTs to remove redundant PHI nodes (well, I'm testing a patch for FRE that will do it already there). Redundant or degenerates which should be propagated? I believe Alan

Re: Remove redundant AND from count reduction loop

2015-06-23 Thread Marc Glisse
On Tue, 23 Jun 2015, Richard Sandiford wrote: +/* Vector comparisons are defined to produce all-one or all-zero results. */ +(simplify + (vec_cond @0 integer_all_onesp@1 integer_zerop@2) + (if (tree_nop_conversion_p (type, TREE_TYPE (@0))) + (convert @0))) I am trying to understand why the

[PATCH/AARCH64] Update ThunderX schedule model

2015-06-23 Thread Andrew Pinski
Hi, This patch updates the schedule model to be more accurate and model SIMD and fp instructions that I had missed out when I had the last patch. OK? Bootstrapped and tested on aarch64-linux-gnu with no regeessions. Thanks, Andrew Pinski ChangeLog: * config/aarch64/thunderx.md

Re: [01/13] Move hash traits to their own header file

2015-06-23 Thread Jeff Law
On 06/16/2015 02:43 AM, Richard Sandiford wrote: This patch just moves the traits classes to their own header file, hash-traits.h. gcc/ * hash-table.h: Include hash-traits.h. (typed_free_remove, typed_noop_remove, pointer_hash, ggc_hasher) (ggc_cache_hasher): Move to...

Re: [03/13] Make ggc_cached_hasher inherit from ggc_hasher

2015-06-23 Thread Jeff Law
On 06/16/2015 02:52 AM, Richard Sandiford wrote: Most of ggc_cache_hasher is duplicated from ggc_hasher. This patch gets the same functionality via inheritance. It also brings the ggc_hasher functions in line with ggc_cache_hasher by making remove() and ggc_mx() take references rather than

Re: [04/13] Add {mark,is}_{empty,deleted} to pointer_hash

2015-06-23 Thread Jeff Law
On 06/16/2015 02:53 AM, Richard Sandiford wrote: This patch just adds the standard pointer handling of empty and deleted entries. As the series goes on, more and more traits classes will inherit these definitions, to the point where we can require the functions to exist. gcc/ *

Re: [PATCH/RFC] Make loop-header-copying more aggressive, rerun before tree-if-conversion

2015-06-23 Thread Jeff Law
On 06/19/2015 11:38 AM, Alan Lawrence wrote: Jeff Law wrote: On 05/22/2015 09:42 AM, Alan Lawrence wrote: This patch does so (and makes slightly less conservative, to tackle the example above). I found I had to make this a separate pass, so that the phi nodes were cleaned up at the end of the

Re: [00/13] Share hash traits between hash_table and hash_set

2015-06-23 Thread Jeff Law
On 06/16/2015 02:42 AM, Richard Sandiford wrote: At the moment we need separate traits classes for hash_table, hash_set and hash_map. I think that's a sign that we don't have the right abstraction. You're probably right. The aim of this series is to unify the traits for hash_table and

Re: [02/13] Replace handle_cache_entry with new interface

2015-06-23 Thread Jeff Law
On 06/16/2015 02:45 AM, Richard Sandiford wrote: As described in the covering note, this patch replaces handle_cache_entry with a new function keep_cache_entry. It also ensures that elements are deleted using the proper function, so that m_n_deleted is updated. I couldn't tell whether the

Re: [05/13] Add nofree_ptr_hash

2015-06-23 Thread Jeff Law
On 06/16/2015 02:55 AM, Richard Sandiford wrote: This patch stops pointer_hash from inheriting typed_noop_remove and instead creates a new class nofree_ptr_hash that inherits from both. It then updates all uses of typed_noop_remove (which are all pointers) and pointer_hash so that they use this

Re: [gomp4.1] Add new versions of GOMP_target{,_data,_update} and GOMP_target_enter_exit_data

2015-06-23 Thread Jakub Jelinek
On Tue, Jun 23, 2015 at 02:40:43PM +0300, Ilya Verbin wrote: On Sat, Jun 20, 2015 at 00:35:14 +0300, Ilya Verbin wrote: Given that a mapped variable in 4.1 can have different kinds across nested data regions, we need to store map-type not only for each var, but also for each structured

RFA: FT32: Fix building gcc.

2015-06-23 Thread Nicholas Clifton
Hi Guys, It seems that the FT32 port of GCC does not have a maintainer at the moment. Nevertheless I have a patch to fix a couple of build time problems compiling gcc for the FT32. Is this OK to apply ? Cheers Nick gcc/ChangeLog 2015-06-23 Nick Clifton ni...@redhat.com *

Re: [C++ Patch] PR 65811

2015-06-23 Thread Jason Merrill
OK. Jason

Re: [PATCH][ARM] PR/65711: Don't pass '-dynamic-linker' when '-shared' is used

2015-06-23 Thread Richard Biener
On Mon, May 18, 2015 at 9:09 PM, Ludovic Courtès l...@gnu.org wrote: Ramana Radhakrishnan ramana@googlemail.com skribis: On Thu, Apr 23, 2015 at 9:29 AM, Ludovic Courtès l...@gnu.org wrote: As discussed at https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65711. Patch is for both 4.8 and 4.9

Re: [PATCH] Check dominator info in compute_dominance_frontiers

2015-06-23 Thread Richard Biener
On Mon, Jun 22, 2015 at 7:10 PM, Tom de Vries tom_devr...@mentor.com wrote: On 22/06/15 13:47, Richard Biener wrote: (eventually also for the case where we end up only computing the fast-query stuff). Like this? ... diff --git a/gcc/dominance.c b/gcc/dominance.c index 9c66ca2..58fc6fd

Re: [PATCH][ARM] PR/65711: Don't pass '-dynamic-linker' when '-shared' is used

2015-06-23 Thread Jakub Jelinek
On Tue, Jun 23, 2015 at 11:30:25AM +0200, Ludovic Courtès wrote: Ramana Radhakrishnan ramana@googlemail.com skribis: On Thu, Apr 23, 2015 at 9:29 AM, Ludovic Courtès l...@gnu.org wrote: As discussed at https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65711. Patch is for both 4.8 and 4.9

Re: [Ping, Patch, fortran, 64674, v3] [OOP] ICE in ASSOCIATE with class array

2015-06-23 Thread Andre Vehreschild
Hi Paul, thanks for the review. Submitted as r224827. Regards, Andre -- Andre Vehreschild * Email: vehre ad gmx dot de Index: gcc/testsuite/gfortran.dg/associate_18.f08 === --- gcc/testsuite/gfortran.dg/associate_18.f08

Re: [PATCH][ARM] PR/65711: Don't pass '-dynamic-linker' when '-shared' is used

2015-06-23 Thread Ludovic Courtès
Ramana Radhakrishnan ramana@googlemail.com skribis: On Thu, Apr 23, 2015 at 9:29 AM, Ludovic Courtès l...@gnu.org wrote: As discussed at https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65711. Patch is for both 4.8 and 4.9 (possibly 5.1 too, I haven’t checked.) OK for trunk. This is also

Re: [i386, PATCH, 3/3] IA MCU psABI support: testsuite.

2015-06-23 Thread Kirill Yukhin
Hello, This patch introduces tests for new psABI. gcc/testsuite/ * gcc.target/i386/iamcu/abi-iamcu.exp: New file. * gcc.target/i386/iamcu/args.h: Likewise. * gcc.target/i386/iamcu/asm-support.S: Likewise. * gcc.target/i386/iamcu/defines.h: Likewise. *

  1   2   >