[PATCH, C++] Fix PR63366: __complex not equivalent to __complex double in C++

2014-09-29 Thread Thomas Preud'homme
According to a comment in grokdeclarator in file gcc/cp/decl.c: /* If we just have complex, it is equivalent to complex double, but if any modifiers at all are specified it is the complex form of TYPE. E.g, complex short is complex short int. */ Yet, __complex is equivalent to

Re: [PATCH i386 AVX512] [57/n] Extend blend/cmp/brodcast insn patterns.

2014-09-29 Thread Uros Bizjak
On Fri, Sep 26, 2014 at 11:04 AM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Hello, Patch in the bottom extends blend/cmp/brodcast insn patterns. Bootstrapped. AVX-512* tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ * config/i386/sse.md

Re: [PATCH i386 AVX512] [58/n] Add vpmul[u]dq insn patterns.

2014-09-29 Thread Uros Bizjak
On Fri, Sep 26, 2014 at 12:33 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Hello, Patch in the bottom adds support for vpmul[u]dq insn patterns. Bootstrapped. AVX-512* tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ * config/i386/sse.md

Re: [PATCH i386 AVX512] [59/n] Add vptest[n]m, ucmp, cmpeq insn patterns.

2014-09-29 Thread Uros Bizjak
On Fri, Sep 26, 2014 at 12:45 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Hello, Patch in the bottom adds support for vptest[n]m, ucmp, cmpeq. Bootstrapped. AVX-512* tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ * config/i386/i386.c

Re: [PATCH i386 AVX512] [60/n] Update 128bit ashrv insn pattern.

2014-09-29 Thread Uros Bizjak
On Fri, Sep 26, 2014 at 1:13 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Hello, This tiny patch extends 128bit ashrv expander. Bootstrapped. AVX-512* tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ * config/i386/sse.md

Re: [PATCH i386 AVX512] [61/n] Update FP logic insn patterns.

2014-09-29 Thread Uros Bizjak
On Fri, Sep 26, 2014 at 2:32 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Hello, This patch extends andnot and any_logic insn patterns. Bootstrapped. AVX-512* tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ * config/i386/sse.md

Re: [PATCH i386 AVX512] [62/n] Add vpmaddubsw,vdbpsadbw insn patterns.

2014-09-29 Thread Uros Bizjak
On Fri, Sep 26, 2014 at 4:09 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Hello, This patch introduces patterns for vpmaddubsw and vdbpsadbw insn. Bootstrapped. AVX-512* tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ * config/i386/sse.md

Re: [Patch, Fortran] Add CO_BROADCAST

2014-09-29 Thread Tobias Burnus
Dominique Dhumieres wrote: The failures for the gfortran.dg/coarray_collectives_9.f90 are fixed with the following patch: Looks good to me. The patch is OK with a ChangLog. Thanks for the patch and sorry for the test fails. Tobias

Re: [PATCH v2] PR libitm/61164: use always_inline consistently

2014-09-29 Thread Jakub Jelinek
On Sat, Sep 27, 2014 at 09:00:00PM +0400, Gleb Fotengauer-Malinovskiy wrote: 2014-09-27 Gleb Fotengauer-Malinovskiy gle...@altlinux.org libitm/ PR libitm/61164 * local_atomic (__always_inline): Add inline. (__calculate_memory_order): Remove inline.

Re: [fortran,patch] Forbid assignment of different character kinds

2014-09-29 Thread FX
It looks like the committee has reversed his opinion on this since the 2008 interp. There is wording in both F2003 and F2008 standards that supports this view, so I’ve closed the PR. Now, here’s a tiny patch to silence the related warning in PR36534. I also remove the condition on

Re: [RFC: Patch, PR 60102] [4.9/4.10 Regression] powerpc fp-bit ices@dwf_regno

2014-09-29 Thread Ulrich Weigand
Maciej W. Rozycki wrote: On Mon, 4 Aug 2014, Edmar wrote: Committed on trunk, revision 213596 Committed on 4.9 branch, revision 213597 This change regressed GDB for e500v2 multilibs, presumably because it does not understand the new DWARF register numbers and does not know how to

Re: [PATCH] Add libstdc++ baseline_symbols for aarch64

2014-09-29 Thread Jonathan Wakely
On 26/09/14 23:42 +0200, Andreas Schwab wrote: Generated by make new-abi-baseline on aarch64-suse-linux. Andreas. * config/abi/post/aarch64-linux-gnu/baseline_symbols.txt: New file. OK, thanks.

Re: Enable TBAA on anonymous types with LTO

2014-09-29 Thread Richard Biener
On Fri, 26 Sep 2014, Jan Hubicka wrote: Hello, this is patch to preserve TBAA for anonymous types to LTO. The difference can be seen on the testcase: namespace { struct A {int a;}; struct B {int b;}; } struct A aa,*a=aa; struct B bb,*b=bb; void setA() { a-a=1; } void

RE: [RFC: Patch, PR 60102] [4.9/4.10 Regression] powerpc fp-bit ices@dwf_regno

2014-09-29 Thread rohitarul...@freescale.com
From: Ulrich Weigand [mailto:uweig...@de.ibm.com] Maciej W. Rozycki wrote: On Mon, 4 Aug 2014, Edmar wrote: Committed on trunk, revision 213596 Committed on 4.9 branch, revision 213597 This change regressed GDB for e500v2 multilibs, presumably because it does not understand

RE: FW: [PATCH] Cilk Keywords (_Cilk_spawn and _Cilk_sync) for C

2014-09-29 Thread Thomas Schwinge
Hi! On Mon, 22 Sep 2014 19:21:33 +, Tannenbaum, Barry M barry.m.tannenb...@intel.com wrote: That's exactly correct. There are two ways to implement a work-stealing scheduler. We refer to them as: [...] Thanks for the explanation. Cilk implements Parent Stealing. Since you're running

[PATCH, ARM] attribute target (thumb,arm)

2014-09-29 Thread Christian Bruel
Hi Ramana, Richard, This patch implements the attribute target (and pragma) to allow function based interworking. as in the updated documentation, the syntax is: __attribute__((target(thumb))) int foo() Forces thumb mode for function foo only. If the file was compiled with -mthumb iit has no

Re: Enable EBX for x86 in 32bits PIC code

2014-09-29 Thread Jakub Jelinek
On Wed, Sep 24, 2014 at 03:20:44PM -0600, Jeff Law wrote: On 09/24/14 14:32, Ilya Enkovich wrote: 2014-09-24 19:27 GMT+04:00 Jeff Law l...@redhat.com: On 09/24/14 00:56, Ilya Enkovich wrote: After register allocation we have no idea where GOT address is and therefore delegitimize_address

Re: [PATCH 2/2] Make -Q --help print param defaults and min/max values

2014-09-29 Thread Richard Biener
On Sat, Sep 27, 2014 at 3:54 PM, Andi Kleen a...@firstfloor.org wrote: From: Andi Kleen a...@linux.intel.com Make -Q --help print the --param default, min, max values, similar to how it does print the defaults for other flags. This is useful to let a option auto tuner automatically query all

Re: [PATCH 1/2] Remove -fshort-double

2014-09-29 Thread Richard Biener
On Sat, Sep 27, 2014 at 3:54 PM, Andi Kleen a...@firstfloor.org wrote: From: Andi Kleen a...@linux.intel.com -fshort-double has crashes the compiler since 4.6 (see PR60410) Since it's an obscure option that apparently nobody uses it the best way to fix it seems to just remove it. This

Re: [PATCH v3] PR libitm/61164: use always_inline consistently

2014-09-29 Thread Gleb Fotengauer-Malinovskiy
2014-09-27 Gleb Fotengauer-Malinovskiy gle...@altlinux.org libitm/ PR libitm/61164 * local_atomic: Rename __always_inline to __libitm_always_inline to eliminate glibc macro redefinition. (__libitm_always_inline): Add inline. (__calculate_memory_order):

Re: [Patch, AArch64] Enable Address sanitizer and UB sanitizer

2014-09-29 Thread Christophe Lyon
On 26 September 2014 23:05, Andreas Schwab sch...@linux-m68k.org wrote: https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=34c65c4 * sanitizer_common/sanitizer_platform_limits_posix.h (__sanitizer___kernel_old_uid_t, __sanitizer___kernel_old_gid_t)

[libstdc++] Refactor python/hook.in

2014-09-29 Thread Siva Chandra
The attached patch refactors python/hook.in so that there are no individual function calls to load pretty printers and xmethods. This was suggested by Tom here: https://gcc.gnu.org/ml/gcc-patches/2014-08/msg02589.html. He indicates that it is better to put as little as possible in the hook file.

Re: [PATCH] Fix finding default baseline symbols directory

2014-09-29 Thread Jonathan Wakely
On 26/09/14 23:42 +0200, Andreas Schwab wrote: Tested on aarch64-suse-linux, where try_cpu=generic. Andreas. * configure.host: Use host_cpu, not try_cpu, to define default abi_baseline_pair. --- libstdc++-v3/configure.host | 4 ++-- 1 file changed, 2 insertions(+), 2

Re: [libstdc++] Refactor python/hook.in

2014-09-29 Thread Jonathan Wakely
On 29/09/14 06:02 -0700, Siva Chandra wrote: The attached patch refactors python/hook.in so that there are no individual function calls to load pretty printers and xmethods. This was suggested by Tom here: https://gcc.gnu.org/ml/gcc-patches/2014-08/msg02589.html. He indicates that it is better

Re: [PATCH v4 1/2] Fix __ENVIRONMENT_MAC_OS_X_VERSION_MIN_REQUIRED__

2014-09-29 Thread FX
Hi Mike, I have not seen any trouble arising following the fix to PR 61407 (https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61407). The patch committed is here: https://gcc.gnu.org/viewcvs?rev=215251root=gccview=rev I’ve just tested the exact same change on the 4.9 branch, and it bootstraps and

Re: [PATCH v3] PR libitm/61164: use always_inline consistently

2014-09-29 Thread Jakub Jelinek
On Mon, Sep 29, 2014 at 04:53:26PM +0400, Gleb Fotengauer-Malinovskiy wrote: 2014-09-27 Gleb Fotengauer-Malinovskiy gle...@altlinux.org Two spaces around name on each side. libitm/ PR libitm/61164 * local_atomic: Rename __always_inline to __libitm_always_inline to

RE: FW: [PATCH] Cilk Keywords (_Cilk_spawn and _Cilk_sync) for C

2014-09-29 Thread Tannenbaum, Barry M
In a nutshell, add the following code to main() before the call to f3(): int status = __cilkrts_set_param(nworkers, 2); if (0 != status) { // Failed to set the number of Cilk workers return status; } Here's the details: There are three sources of information the Cilk

[PATCH] Don't call fatal_error before error reporting has been initialized.

2014-09-29 Thread Ilya Tocar
Hi, Currently if call to atexit (lto_wrapper_cleanup) fails we won't report error as we haven't initialized error-reporting infrastructure. This patch moves this call after diagnostic_initialize. I hope that we can't exit inside diagnostic_initialize. Otherwise we won't cleanup after it. Ok for

Re: [PATCH] Redesign jump threading profile updates

2014-09-29 Thread Teresa Johnson
On Fri, Aug 1, 2014 at 10:10 PM, Teresa Johnson tejohn...@google.com wrote: On Wed, Jul 23, 2014 at 2:08 PM, Teresa Johnson tejohn...@google.com wrote: On Tue, Jul 22, 2014 at 7:29 PM, Jeff Law l...@redhat.com wrote: On 03/26/14 17:44, Teresa Johnson wrote: Recently I discovered that the

RE: FW: [PATCH] Cilk Keywords (_Cilk_spawn and _Cilk_sync) for C

2014-09-29 Thread Thomas Schwinge
Hi! On Mon, 29 Sep 2014 13:58:31 +, Tannenbaum, Barry M barry.m.tannenb...@intel.com wrote: In a nutshell, add the following code to main() before the call to f3(): int status = __cilkrts_set_param(nworkers, 2); if (0 != status) { // Failed to set the number of Cilk

[patch] Update libstdc++ status table.

2014-09-29 Thread Jonathan Wakely
Update the docs again. Committed to trunk. commit ef8c7f18cdfc087d54b0bbe2a1cd171409eb1f18 Author: Jonathan Wakely jwak...@redhat.com Date: Mon Sep 29 15:13:33 2014 +0100 * doc/xml/manual/status_cxx2011.xml: Update. * doc/html/manual/status.html: Regenerate. diff --git

Re: [Patch, Fortran] Add CO_BROADCAST

2014-09-29 Thread Tobias Burnus
On Mon, Sep 29, 2014 at 10:17:04AM +0200, Tobias Burnus wrote: Dominique Dhumieres wrote: The failures for the gfortran.dg/coarray_collectives_9.f90 are fixed with the following patch: Looks good to me. The patch is OK with a ChangLog. Actually, I missed the following part: ... - call

Re: [libstdc++] Refactor python/hook.in

2014-09-29 Thread Tom Tromey
Jonathan I'll commit it later this week unless I hear objections from Jonathan Tom. It looks reasonable to me. Tom

[c++-concepts] Check function concept definitions

2014-09-29 Thread Andrew Sutton
This fixes an ICE trying to normalize a function concept with multiple statements. That error will now be diagnosed at the point of definition. Jason, do you want to review this before I commit? This is a pretty small patch. 2014-09-01 Andrew Sutton andrew.n.sut...@gmail.com Check

RE: FW: [PATCH] Cilk Keywords (_Cilk_spawn and _Cilk_sync) for C

2014-09-29 Thread Tannenbaum, Barry M
Looks good to me. I apologize for spamming you with the extra details. I wasn't clear that this was a patch instead of a bug report. I believe that Igor is the one who controls the GCC submission for Cilk Plus. - Barry -Original Message- From: Thomas Schwinge

Re: [PATCH 1/2] Remove -fshort-double

2014-09-29 Thread Andi Kleen
As we saw LTO fixes for -fshort-double it's clear that this flag _is_ used for some embedded archs. Did we? It has been ICEing since 4.5, which is before LTO. -Andi

Re: [PATCH 1/2] Remove -fshort-double

2014-09-29 Thread Jakub Jelinek
On Mon, Sep 29, 2014 at 08:20:03AM -0700, Andi Kleen wrote: As we saw LTO fixes for -fshort-double it's clear that this flag _is_ used for some embedded archs. Did we? It has been ICEing since 4.5, which is before LTO. Depends on which target. Aren't the ICEs i?86/x86_64 backend ICEs?

Re: [c++-concepts] Check function concept definitions

2014-09-29 Thread Jason Merrill
On 09/29/2014 10:55 AM, Andrew Sutton wrote: Jason, do you want to review this before I commit? This is a pretty small patch. No need to wait for review before committing to the branch. + // If fn was declared with auto, make sure the result type is bool. + if (FNDECL_USED_AUTO (fn)

Re: [PATCH 1/2] Remove -fshort-double

2014-09-29 Thread Richard Biener
On September 29, 2014 5:21:36 PM CEST, Jakub Jelinek ja...@redhat.com wrote: On Mon, Sep 29, 2014 at 08:20:03AM -0700, Andi Kleen wrote: As we saw LTO fixes for -fshort-double it's clear that this flag _is_ used for some embedded archs. Did we? It has been ICEing since 4.5, which is before

Re: [PATCH C++] - SD-6 Implementation Part 1 - __has_include.

2014-09-29 Thread Ed Smith-Rowland
On 09/25/2014 12:57 PM, Jason Merrill wrote: On 09/01/2014 09:34 PM, Ed Smith-Rowland wrote: (open_file_failed()): Not an error to not find a header file for __has_include__. Hmm, looks like this means that __has_include__ will silently return false if a header exists but is

Re: [PATCH v3] PR libitm/61164: use always_inline consistently

2014-09-29 Thread Torvald Riegel
On Mon, 2014-09-29 at 16:53 +0400, Gleb Fotengauer-Malinovskiy wrote: -#undef __always_inline -#define __always_inline __attribute__((always_inline)) +#define __libitm_always_inline inline __attribute__((always_inline)) The previous code seems to work in libstdc++. I believe that eventually,

Re: Enable TBAA on anonymous types with LTO

2014-09-29 Thread Jan Hubicka
Why not just make all anonymous types their own canonical type? (of course considering type variants) If C++ FE sets canonical type always to main variant, it should work. Is it always the case? I noticed you do this for variadic types. I tought there is reason why canonical types differ from

Re: Enable TBAA on anonymous types with LTO

2014-09-29 Thread Jan Hubicka
Why not just make all anonymous types their own canonical type? (of course considering type variants) If C++ FE sets canonical type always to main variant, it should work. Is it always the case? I noticed you do this for variadic types. I tought there is reason why canonical types

Re: [c++-concepts] Check function concept definitions

2014-09-29 Thread Andrew Sutton
Hmm, have we actually discussed this in core review? I'm not seeing it on the wiki. Constexpr started out this way too, and allowing static_assert was added pretty fast. C++11 said its function-body shall be = delete, = default, or a compound-statement that contains only — null

Re: [PATCH 1/2] Remove -fshort-double

2014-09-29 Thread Andi Kleen
So - no, you can't simply remove it. But IMHO it should become a target-specific flag. How about a patch to just disable it for x86? -Andi

Re: FW: [PATCH] Cilk Keywords (_Cilk_spawn and _Cilk_sync) for C

2014-09-29 Thread Jeff Law
On 09/29/14 08:26, Thomas Schwinge wrote: Hi! On Mon, 29 Sep 2014 13:58:31 +, Tannenbaum, Barry M barry.m.tannenb...@intel.com wrote: In a nutshell, add the following code to main() before the call to f3(): int status = __cilkrts_set_param(nworkers, 2); if (0 != status) {

Re: [PATCH v3] PR libitm/61164: use always_inline consistently

2014-09-29 Thread Jakub Jelinek
On Mon, Sep 29, 2014 at 05:35:24PM +0200, Torvald Riegel wrote: On Mon, 2014-09-29 at 16:53 +0400, Gleb Fotengauer-Malinovskiy wrote: -#undef __always_inline -#define __always_inline __attribute__((always_inline)) +#define __libitm_always_inline inline __attribute__((always_inline)) The

Commit: MSP430: Improve -mhwmult= opton and fix prologues and epilogues for naked functions

2014-09-29 Thread Nick Clifton
Hi Guys, I am applying the patch below to the MSP430 backend. It improves the -mhwmult=auto command line option so that MCUs without any hardware support will be recognised if the -mmcu= option has also been given. The patch also fixes a small problem with the prologue and epilogue

Re: [PATCH] PR libitm/61164: redefinition of __always_inline

2014-09-29 Thread Gleb Fotengauer-Malinovskiy
2014-09-27 Gleb Fotengauer-Malinovskiy gle...@altlinux.org libitm/ PR libitm/61164 * local_atomic (__always_inline): Rename to... (__libitm_always_inline): ... this. --- On Mon, Sep 29, 2014 at 03:38:25PM +0200, Jakub Jelinek wrote: Why do you want to add inline

[jit] Documentation tweaks

2014-09-29 Thread David Malcolm
Committed to branch dmalcolm/jit: gcc/jit/ChangeLog.jit: * TODO.rst: Update. * docs/topics/expressions.rst (gcc_jit_context_new_call): Add a note clarifying the behavior of this entrypoint. * docs/topics/functions.rst (Creating and using functions): Markup

Re: [c++-concepts] Check function concept definitions

2014-09-29 Thread Jason Merrill
On 09/29/2014 11:46 AM, Andrew Sutton wrote: The main reason for the restriction is that concept definitions are normalized into a single constraint-expression. And it's not obvious how things like using declarations and static-assertions should be interpreted within the constraint language. A

Re: [PATCH v3] PR libitm/61164: use always_inline consistently

2014-09-29 Thread Torvald Riegel
On Mon, 2014-09-29 at 17:38 +0200, Jakub Jelinek wrote: On Mon, Sep 29, 2014 at 05:35:24PM +0200, Torvald Riegel wrote: On Mon, 2014-09-29 at 16:53 +0400, Gleb Fotengauer-Malinovskiy wrote: -#undef __always_inline -#define __always_inline __attribute__((always_inline)) +#define

Re: [PATCH] microblaze: microblaze.md: Use VOID instead of SI to fix ((void (*)(void)) 0)() issue

2014-09-29 Thread Michael Eager
On 09/25/14 07:03, Chen Gang wrote: 2014-09-25 Chen Ganggang.chen.5...@gmail.com gcc: * config/microblaze/microblaze.md (call_internal1): Use VOID instead of SI to fix ((void (*)(void)) 0)() issue gcc/testsuite/: 2014-09-28 Chen Gang gang.chen.5...@gmail.com *

Re: [PATCH] combine: Allow substituting the target reg of a clobber

2014-09-29 Thread Jeff Law
On 09/27/14 16:03, Segher Boessenkool wrote: On Mon, Sep 22, 2014 at 04:20:12PM -0600, Jeff Law wrote: Can you add a testcase which shows the 3-insn combination from PR62151 applying? I've tried to make a stable future-proof testcase that does such a three-insn combination. Not easy at all.

Re: [PATCH] Fix ICE in redirect_jump_1 (PR inline-asm/63282)

2014-09-29 Thread Jeff Law
On 09/28/14 23:36, Jakub Jelinek wrote: Hi! On the following testcase, dead_or_predicable decides to call redirect_jump_1 on asm goto which has more than one label, but the bb still has just two successors (one of the labels points to code label at the start of the fallthru bb) and

Re: [PATCH v4 1/2] Fix __ENVIRONMENT_MAC_OS_X_VERSION_MIN_REQUIRED__

2014-09-29 Thread Mike Stump
On Sep 29, 2014, at 6:20 AM, FX fxcoud...@gmail.com wrote: I have not seen any trouble arising following the fix to PR 61407 (https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61407). The patch committed is here: https://gcc.gnu.org/viewcvs?rev=215251root=gccview=rev I’ve just tested the

Re: [PATCH] [ARM] [RFC] Fix longstanding push_minipool_fix ICE (PR49423, lp1296601)

2014-09-29 Thread Charles Baylis
Given that we've had this bake sufficiently on trunk and have seen no regressions reported it should be fine to go back to these branches. Further we've had the 4.9.1 and 4.8.3 releases recently so I'd say Yes, unless the RM's object in the next 24 hours. Make check passed on 4.8 and 4.9

[PATCHv3][PING] Enable -fsanitize-recover for KASan

2014-09-29 Thread Yury Gribov
Hi all, This patch enables -fsanitize-recover for KASan by default. This causes KASan to continue execution after error in case of inline instrumentation. This feature is needed because - reports during early bootstrap won't even be printed - needed to run all tests w/o rebooting machine for

[PATCHv3][Kasan][PING] Allow to override Asan shadow offset from command line

2014-09-29 Thread Yury Gribov
Hi all, Kasan developers has asked for an option to override offset of Asan shadow memory region. This should simplify experimenting with memory layouts on 64-bit architectures. New patch which checks that -fasan-shadow-offset is only enabled for -fsanitize=kernel-address. I (unfortunately)

[patch] Flatten function.h take 2

2014-09-29 Thread Andrew MacLeod
On 09/16/2014 05:23 PM, Andrew MacLeod wrote: On 09/16/2014 05:12 PM, Joseph S. Myers wrote: On Tue, 16 Sep 2014, Andrew MacLeod wrote: I did an include file reduction on all the language/*.[ch] and core *.[ch] files, but left the target files with the full complement of 7 includes that

Re: [PATCH] Fix finding default baseline symbols directory

2014-09-29 Thread Andreas Schwab
Jonathan Wakely jwak...@redhat.com writes: Would a safer change be to just add a new pattern for aarch64? --- a/libstdc++-v3/configure.host +++ b/libstdc++-v3/configure.host @@ -345,6 +345,9 @@ case ${host} in x86_64) abi_baseline_pair=x86_64-linux-gnu ;; +

RE: [RFC: Patch, PR 60102] [4.9/4.10 Regression] powerpc fp-bit ices@dwf_regno

2014-09-29 Thread Maciej W. Rozycki
On Mon, 29 Sep 2014, rohitarul...@freescale.com wrote: As I understand it, the change was supposed to only affect GCC internals, all externally generated debug info was supposed to remain unchanged. If there are changes in debug info, something must have gone wrong. Let me check if

[PATCH, rs6000] Remove splat calls with out-of-range arguments from gcc.dg/vmx/ops.c

2014-09-29 Thread Bill Schmidt
Hi, While working on another patch, I observed that the test case gcc.dg/vmx/ops.c contains numerous calls to vec_splat and friends for which the second argument (the element selector) is out of range. At best these calls are invalid; as it is, we generate insns that can cause trouble during

Re: [PATCH 2/n] OpenMP 4.0 offloading infrastructure: LTO streaming

2014-09-29 Thread Ilya Verbin
On 29 Sep 03:10, Jan Hubicka wrote: dump for me implied debug dump. LTO is usually called streaming, so prehaps need_lto_stremaing? Fixed. +initialize_offload (void) Perhaps have_offload_p? Nothing is initialized here... The next patch will add some initialization to this function. And

Re: [PATCHv3][PING] Enable -fsanitize-recover for KASan

2014-09-29 Thread Jakub Jelinek
On Mon, Sep 29, 2014 at 09:21:11PM +0400, Yury Gribov wrote: This patch enables -fsanitize-recover for KASan by default. This causes KASan to continue execution after error in case of inline instrumentation. This feature is needed because - reports during early bootstrap won't even be printed

Re: [PATCH, rs6000] Remove splat calls with out-of-range arguments from gcc.dg/vmx/ops.c

2014-09-29 Thread David Edelsohn
On Mon, Sep 29, 2014 at 1:27 PM, Bill Schmidt wschm...@linux.vnet.ibm.com wrote: Hi, While working on another patch, I observed that the test case gcc.dg/vmx/ops.c contains numerous calls to vec_splat and friends for which the second argument (the element selector) is out of range. At best

Re: [c++-concepts] Check function concept definitions

2014-09-29 Thread Roland Bock
On 2014-09-29 18:32, Jason Merrill wrote: On 09/29/2014 11:46 AM, Andrew Sutton wrote: The main reason for the restriction is that concept definitions are normalized into a single constraint-expression. And it's not obvious how things like using declarations and static-assertions should be

Re: [PATCH x86_64] Optimize access to globals in -fpie -pie builds with copy relocations

2014-09-29 Thread Sriraman Tallam
Ping. On Fri, Sep 19, 2014 at 2:11 PM, Sriraman Tallam tmsri...@google.com wrote: Hi Richard, I also ran the gcc testsuite with RUNTESTFLAGS=--tool_opts=-mcopyrelocs to check for issues. The only test that failed was g++.dg/tsan/default_options.C. It uses -fpie -pie and BFD ld to link.

Re: [Patch AArch64] Fix extended register width

2014-09-29 Thread Carrot Wei
Ping. On Mon, Sep 22, 2014 at 11:41 AM, Carrot Wei car...@google.com wrote: Hi The extended register width in add/adds/sub/subs/cmp instructions is not always the same as target register, it depends on both target register width and extension type. But in current implementation the extended

Re: __intN patch 3/5: main __int128 - __intN conversion.

2014-09-29 Thread DJ Delorie
Just one question about the include/std/limits changes below. It seems that __glibcxx_signed_b isn't strictly necessary as it doesn't use the B argument, so is it just there for consistency? Yup.

Re: [shrink-wrap] should not sink instructions which may cause trap ?

2014-09-29 Thread Jiong Wang
On 26/09/14 17:12, Jeff Law wrote: On 09/26/14 08:50, Jiong Wang wrote: if (may_trap_p (x)) don't sink this instruction. any comments? Should be checking if x may throw internally instead. Richard, thanks for the suggestion, have used insn_could_throw_p to do the check,

[PATCH] PR63404, gcc 5 miscompiles linux block layer

2014-09-29 Thread Jiong Wang
it's exposed by linux kernel for x86. the root cause is current single_set will ignore CLOBBER USE, while we need to take them into account when handling shrink-wrap. this patch add one parameter to single_set_2 to support return NULL_RTX if we want to remove any side-effect. add a new helper

libcilkrts: Remove unused function __cilkrts_sysdep_is_worker_thread_id (was: Cilk Library)

2014-09-29 Thread Thomas Schwinge
Hi! On Wed, 9 Oct 2013 18:32:11 +, Iyer, Balaji V balaji.v.i...@intel.com wrote: [libcilkrts] I have found a function that is -- as far as I can tell -- unused, and I'm thus proposing to remove it. This increases portability, as this code has dependencies on the operating system. Tested

libcilkrts: Use AC_USE_SYSTEM_EXTENSIONS (was: Cilk Library)

2014-09-29 Thread Thomas Schwinge
Hi! On Wed, 9 Oct 2013 18:32:11 +, Iyer, Balaji V balaji.v.i...@intel.com wrote: [libcilkrts] Here is a patch to have libcilkrts use AC_USE_SYSTEM_EXTENSIONS (as other libraries are doing) instead of manually fiddling with the _GNU_SOURCE definition. This increases portability, as most of

libcilkrts: GNU toolchain, GNU linker scripts (was: Cilk Library)

2014-09-29 Thread Thomas Schwinge
Hi! On Wed, 9 Oct 2013 18:32:11 +, Iyer, Balaji V balaji.v.i...@intel.com wrote: [libcilkrts] As requested during patch review, symbol versioning infrastructure has been added to libcilkrts. However, this is currently described/implemented as Linux-only, while in fact it's standard GNU

libcilkrts: GNU Hurd port, and some code cleanup/consolidation (was: Cilk Library)

2014-09-29 Thread Thomas Schwinge
Hi! On Wed, 9 Oct 2013 18:32:11 +, Iyer, Balaji V balaji.v.i...@intel.com wrote: [libcilkrts] Currently, by means of the libcilkrts/configure.tgt file that has been added during patch review, libcilkrts is attempted to be built for all *-*-gnu* system, but it has actually only been ported

RE: libcilkrts: Remove unused function __cilkrts_sysdep_is_worker_thread_id (was: Cilk Library)

2014-09-29 Thread Tannenbaum, Barry M
It's a remnant from something we were attempting to support (and abandoned) 4 years ago. I'm fine with removing it from the runtime. Igor, I'll make the change and send you a new copy. - Barry -Original Message- From: Thomas Schwinge [mailto:tho...@codesourcery.com] Sent: Monday,

Re: [PATCH] PR63404, gcc 5 miscompiles linux block layer

2014-09-29 Thread Richard Henderson
On 09/29/2014 11:12 AM, Jiong Wang wrote: +inline rtx single_set_no_clobber_use (const rtx_insn *insn) +{ + if (!INSN_P (insn)) +return NULL_RTX; + + if (GET_CODE (PATTERN (insn)) == SET) +return PATTERN (insn); + + /* Defer to the more expensive case, and return NULL_RTX if

Re: __intN patch 3/5: main __int128 - __intN conversion.

2014-09-29 Thread Jonathan Wakely
On 29/09/14 14:06 -0400, DJ Delorie wrote: Just one question about the include/std/limits changes below. It seems that __glibcxx_signed_b isn't strictly necessary as it doesn't use the B argument, so is it just there for consistency? Yup. OK, thanks for confirming.

Re: [PATCH] Fix finding default baseline symbols directory

2014-09-29 Thread Jonathan Wakely
On 29/09/14 19:24 +0200, Andreas Schwab wrote: Jonathan Wakely jwak...@redhat.com writes: Would a safer change be to just add a new pattern for aarch64? --- a/libstdc++-v3/configure.host +++ b/libstdc++-v3/configure.host @@ -345,6 +345,9 @@ case ${host} in x86_64)

[debug-early] rearrange some checks in gen_subprogram_die

2014-09-29 Thread Aldy Hernandez
I'm rearranging some code in Michael's original patch to minimize the difference with mainline. It seems that the check for DECL_STRUCT_FUNCTION (decl)-gimple_df, was merely a check to see if we had already set the FDE bits for the decl in question. I've moved the check inside the original

Re: [PATCH] PR63404, gcc 5 miscompiles linux block layer

2014-09-29 Thread Jiong Wang
On 29/09/14 19:32, Richard Henderson wrote: On 09/29/2014 11:12 AM, Jiong Wang wrote: +inline rtx single_set_no_clobber_use (const rtx_insn *insn) +{ + if (!INSN_P (insn)) +return NULL_RTX; + + if (GET_CODE (PATTERN (insn)) == SET) +return PATTERN (insn); + + /* Defer to the more

Add myself as libstdc++ special modes maintainer

2014-09-29 Thread François Dumont
I added myself as libstdc++ special modes maintainer. Special modes are debug, profile and parallel modes. Thanks for your trust. François

[PATCH, rs6000, libcpp] Revise search_line_fast to avoid old unaligned load sequences

2014-09-29 Thread Bill Schmidt
Hi, The vec_lvsl and vec_lvsr interfaces are deprecated for little-endian Power, and really should not be used on big-endian Power either when the target CPU is power8 or above. The lexer in libcpp currently makes use of these interfaces in search_line_fast(). This patch provides a new version

Re: [PATCH] PR63404, gcc 5 miscompiles linux block layer

2014-09-29 Thread David Malcolm
On Mon, 2014-09-29 at 20:24 +0100, Jiong Wang wrote: On 29/09/14 19:32, Richard Henderson wrote: On 09/29/2014 11:12 AM, Jiong Wang wrote: +inline rtx single_set_no_clobber_use (const rtx_insn *insn) +{ + if (!INSN_P (insn)) +return NULL_RTX; + + if (GET_CODE (PATTERN (insn))

Re: libcilkrts: Remove unused function __cilkrts_sysdep_is_worker_thread_id

2014-09-29 Thread Jeff Law
On 09/29/14 12:13, Thomas Schwinge wrote: Hi! On Wed, 9 Oct 2013 18:32:11 +, Iyer, Balaji V balaji.v.i...@intel.com wrote: [libcilkrts] I have found a function that is -- as far as I can tell -- unused, and I'm thus proposing to remove it. This increases portability, as this code has

Re: libcilkrts: Remove unused function __cilkrts_sysdep_is_worker_thread_id

2014-09-29 Thread Thomas Schwinge
Hi! On Mon, 29 Sep 2014 15:00:03 -0600, Jeff Law l...@redhat.com wrote: On 09/29/14 12:13, Thomas Schwinge wrote: libcilkrts/ * [...] The Cilk+ runtime is shared with ICC and they'll need to pull in this code first. We can then pick it up via merges. Yeah, Barry

Re: [PATCHv3][PING] Enable -fsanitize-recover for KASan

2014-09-29 Thread Konstantin Serebryany
+Alexey Samsonov On Mon, Sep 29, 2014 at 10:43 AM, Jakub Jelinek ja...@redhat.com wrote: On Mon, Sep 29, 2014 at 09:21:11PM +0400, Yury Gribov wrote: This patch enables -fsanitize-recover for KASan by default. This causes KASan to continue execution after error in case of inline

Re: [PATCH IRA] update_equiv_regs fails to set EQUIV reg-note for pseudo with more than one definition

2014-09-29 Thread Jeff Law
On 09/27/14 08:48, Felix Yang wrote: Thanks for the explaination. I have changed the loop_depth into a short interger hoping that we can save some memory :-) Thanks. Attached please find the updated patch. Bootstrapped and reg-tested on x86_64-suse-linux. Please do a final revew once the

[PATCH, rs6000] Warn for deprecated use of vec_lvsl and vec_lvsr for little endian

2014-09-29 Thread Bill Schmidt
Hi, The vec_lvsl and vec_lvsr interfaces are deprecated for little endian in the ELFv2 ABI document. At the moment, these interfaces will produce incorrect code, and the only indication a programmer has of this is that his or her code does not function correctly. This patch adds a warning

Re: [Patch, Fortran] Add CO_BROADCAST

2014-09-29 Thread Dominique d'Humières
Le 29 sept. 2014 à 23:56, Dominique d'Humières domi...@lps.ens.fr a écrit : Unless there is an objection I plan to commit tomorrow the following patch with a change log: --- ../_clean/gcc/testsuite/gfortran.dg/coarray_collectives_9.f90 2014-09-25 12:14:05.0 +0200 +++

[PATCH, rs6000] Generate LE code for vec_lvsl and vec_lvsr that is compatible with BE code

2014-09-29 Thread Bill Schmidt
Hi, Up till now we have not attempted to generate code for LE usage of vec_lvsl and vec_lvsr that is compatible with expected BE usage. The LE code sequence corresponding to lvsl/vperm is not good, and we encourage programmers to convert those sequences to use direct assignment and the type

Re: [PATCHv3][PING] Enable -fsanitize-recover for KASan

2014-09-29 Thread Alexey Samsonov
(resending in plain-text mode) -fasan-recover doesn't look like a good idea - for instance, in Clang, we never use ?san in flag names, preferring -fsanitize-whatever. What's the rationale behind splitting -fsanitize-recover in two flags (ASan- and UBSan- specific)? Is there no way to keep a

Re: [RFC] optimize x - y cmp 0 with undefined overflow

2014-09-29 Thread Eric Botcazou
Yeah, that sounds good to me. Here's what I have at last commited after testing on x86-64/Linux. 2014-09-29 Eric Botcazou ebotca...@adacore.com * tree-vrp.c (get_single_symbol): New function. (build_symbolic_expr): Likewise. (symbolic_range_based_on_p): New

Re: [PATCHv3][PING] Enable -fsanitize-recover for KASan

2014-09-29 Thread Jakub Jelinek
On Mon, Sep 29, 2014 at 03:36:20PM -0700, Alexey Samsonov wrote: -fasan-recover doesn't look like a good idea - for instance, in Clang, we never use ?san in flag names, preferring -fsanitize-whatever. What's the rationale behind splitting -fsanitize-recover in two flags (ASan- and UBSan-

[patch commit] [SH] Use define_c_enum in sh.md

2014-09-29 Thread Kaz Kojima
I've noticed that config/sh/sh.md uses define_constants to define unspec and unspecv numbers, though config/sh/sync.md uses define_c_enum for them. This causes collisions of some numbers. The attached patch would be an obvious fix. Tested on sh4-unknown-linux. Applied to trunk. I'll backport

Re: [PATCHv3][PING] Enable -fsanitize-recover for KASan

2014-09-29 Thread Alexey Samsonov
On Mon, Sep 29, 2014 at 4:17 PM, Jakub Jelinek ja...@redhat.com wrote: On Mon, Sep 29, 2014 at 03:36:20PM -0700, Alexey Samsonov wrote: -fasan-recover doesn't look like a good idea - for instance, in Clang, we never use ?san in flag names, preferring -fsanitize-whatever. What's the rationale

libgo patch committed: Use -Qunused-arguments with asm tests

2014-09-29 Thread Ian Lance Taylor
This patch to the libgo configure script changes it to use -Qunused-arguments when running the assembler tests. Apparently clang by default complains when arguments to the driver are unused, as in -I arguments when invoked on a .s file. Passing -Qunused-arguments disables these warnings, and

libffi patch RFA: Pass -Qunused-arguments for asm files

2014-09-29 Thread Ian Lance Taylor
Similar to a recent patch to libgo, this patch to the libffi configure script checks whether the compiler support -Qunused-arguments. If it does, it passes -Qunused-arguments when invoking the compiler on .s files. This is because the clang driver complains by default when given useless

Re: [PATCHv3][PING] Enable -fsanitize-recover for KASan

2014-09-29 Thread Konstantin Serebryany
On Mon, Sep 29, 2014 at 4:26 PM, Alexey Samsonov samso...@google.com wrote: On Mon, Sep 29, 2014 at 4:17 PM, Jakub Jelinek ja...@redhat.com wrote: On Mon, Sep 29, 2014 at 03:36:20PM -0700, Alexey Samsonov wrote: -fasan-recover doesn't look like a good idea - for instance, in Clang, we never

  1   2   >