[PATCH, ifcvt] Allow CC mode if HAVE_cbranchcc4 (fix s390 build)

2014-11-05 Thread Zhenqiang Chen
Hi, The patch add runtime check to fix s390 build fail (https://gcc.gnu.org/ml/gcc-patches/2014-11/msg00050.html). And there is additional code to workaround s390 cstorecc4 issue. Bootstrap and no make check regression on X86-64. Build s390-linux-gnu and s390x-linux-gnu. I do not have env to ru

Re: Add uniform_inside_sphere_distribution

2014-11-05 Thread Marc Glisse
On Wed, 5 Nov 2014, Ed Smith-Rowland wrote: On 11/05/2014 04:25 PM, Marc Glisse wrote: On Wed, 5 Nov 2014, Ed Smith-Rowland wrote: Like the uniform_on_sphere_distribution which is used inside, the 2-dimensional case uses rejection Could you point out where in the code you are special-casing

Re: [PATCH 00/13] Go closures, libffi, and the static chain

2014-11-05 Thread Richard Henderson
On 11/05/2014 10:33 PM, Lynn A. Boger wrote: > What about the libffi changes that are needed to make this work on other > platforms, like PowerPC? I've been working my way through the currently supported libgo targets on libffi-discuss, hoping to get them in upstream libffi before importing to gcc

[committed] MAINTAINERS (Write After Approval): Add myself

2014-11-05 Thread Prachi Godbole
2014-11-06 Prachi Godbole * MAINTAINERS (Write After Approval): Add myself. Index: MAINTAINERS === --- MAINTAINERS (revision 217171) +++ MAINTAINERS (working copy) @@ -395,6 +395,7 @@ Tristan Gingold

Re: [PATCH 3/5] IPA ICF pass

2014-11-05 Thread Joey Ye
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63747 is likely caused by this patch. compare_gimple_switch does not check CASE_LOW and CASE_HIGH, resulting merging functions not identical. Interestingly in the first a few versions of this patch CASE_LOW/HIGH were checked. But last versions only chec

[PING 2][PATCH, AArch64] Add support for -mlong-calls option

2014-11-05 Thread Yangfei (Felix)
Hello, Ping for https://gcc.gnu.org/ml/gcc-patches/2014-10/msg02933.html Thanks

[PATCH PR 63758] fix liblto_plugin.so undefined _environ reference on OSX host

2014-11-05 Thread Roland McGrath
As I described in the bug, OSX (as documented) does not permit shared libraries to refer directly to the environ global variable. Instead, they must call the function _NSGetEnviron to get the address of environ. This becomes a problem when some libiberty code that uses environ directly gets linke

Re: [PATCH] PR36312

2014-11-05 Thread Manuel López-Ibáñez
On 5 November 2014 21:57, Eric Botcazou wrote: >> 2014-10-25 Anthony Brandon >> >> PR driver/36312 >> * diagnostic-core.h: Add prototype for fatal_error. >> * diagnostic.c (fatal_error): New function fatal_error. >> * gcc.c (store_arg): Remove have_o_argbuf_index. >> (proces

Re: [PATCH] PR36312

2014-11-05 Thread Manuel López-Ibáñez
On 6 November 2014 00:33, Joseph Myers wrote: > On Wed, 5 Nov 2014, Manuel López-Ibáñez wrote: > >> Sorry for the breakage. I guess Anthony didn't configure with >> --enable-languages=all,ada. It is a typical mistake I have myself done >> in the past. Since the fix is not obvious to me and I'm not

Re: [PATCH] Don't segv on __atomic_store (PR c/61553)

2014-11-05 Thread Matthias Klose
is this ok to backport to 4.9? testsuite passes without regressions with this patch on the 4.9 branch. Matthias Am 23.06.2014 um 20:21 schrieb Marek Polacek: > On Mon, Jun 23, 2014 at 04:39:55PM +0200, Marek Polacek wrote: >> --- gcc/testsuite/c-c++-common/pr61553.c >> +++ gcc/testsuite/c-c++-c

[COMMITTED][MIPS] Fix all test failures for implicit declaration and int warnings

2014-11-05 Thread Matthew Fortune
This patch fixes all failures in the MIPS target tests which were failing because of -Wimplicit-int or -Wimplicit-function-declaration. +2014-11-05 Matthew Fortune + + * gcc.target/mips/asm-1.c (bar): Add prototype. + * gcc.target/mips/call-1.c (f, h): Add return type. + * gc

Re: Add uniform_inside_sphere_distribution

2014-11-05 Thread Ed Smith-Rowland
On 11/05/2014 04:25 PM, Marc Glisse wrote: On Wed, 5 Nov 2014, Ed Smith-Rowland wrote: Like the uniform_on_sphere_distribution which is used inside, the 2-dimensional case uses rejection Could you point out where in the code you are special-casing dimension 2? Somehow I can't see it in the p

Re: [PATCH] PR36312

2014-11-05 Thread Joseph Myers
On Wed, 5 Nov 2014, Manuel López-Ibáñez wrote: > Sorry for the breakage. I guess Anthony didn't configure with > --enable-languages=all,ada. It is a typical mistake I have myself done > in the past. Since the fix is not obvious to me and I'm not sure how > much time Anthony would require to find a

Re: [AArch64, Patch] Restructure arm_neon.h vector types's implementation(Take 2).

2014-11-05 Thread James Greenhalgh
On Wed, Nov 05, 2014 at 09:50:52PM +, Marc Glisse wrote: > On Wed, 5 Nov 2014, Tejas Belagod wrote: > > >>> 2014-10-01 Tejas Belagod > >>> > >>> * config/aarch64/aarch64-builtins.c > >>> (aarch64_build_scalar_type): Remove. > >>> (aarch64_scalar_builtin_types, aar

Re: [PATCH, Pointer Bounds Checker 19/x] Support bounds in expand

2014-11-05 Thread Eric Botcazou
> IMHO it does. That or introducing a new store_expr_with_bounds (with the > new argument) and letting store_expr be a wrapper for that, passing the > NULL. Basically anything that avoids adding a new parameter for most of > the existing calls to store_expr. That looks so C-ish though... Can't

Re: [PATCH] PR36312

2014-11-05 Thread Manuel López-Ibáñez
On 5 November 2014 23:39, Eric Botcazou wrote: >> Sorry for the breakage. I guess Anthony didn't configure with >> --enable-languages=all,ada. It is a typical mistake I have myself done >> in the past. Since the fix is not obvious to me and I'm not sure how >> much time Anthony would require to fi

Re: [gomp4] Use GOMP_OFFLOAD_ prefix for (OpenACC) plugin hooks

2014-11-05 Thread Ilya Verbin
2014-11-06 0:28 GMT+03:00 Julian Brown : > I made some changes to table initialisation on the gomp4 branch also -- > probably not enough to genuinely support multiple devices, but > hopefully some of the way there. Have you seen those? I haven't > considered dlopened libs though. > > Similarly, I a

Re: [PATCH] PR36312

2014-11-05 Thread Eric Botcazou
> Sorry for the breakage. I guess Anthony didn't configure with > --enable-languages=all,ada. It is a typical mistake I have myself done > in the past. Since the fix is not obvious to me and I'm not sure how > much time Anthony would require to find a fix, I reverted the patch > for now. Thanks.

Re: [PATCH] PR36312

2014-11-05 Thread Manuel López-Ibáñez
On 5 November 2014 21:57, Eric Botcazou wrote: >> 2014-10-25 Anthony Brandon >> >> PR driver/36312 >> * diagnostic-core.h: Add prototype for fatal_error. >> * diagnostic.c (fatal_error): New function fatal_error. >> * gcc.c (store_arg): Remove have_o_argbuf_index. >> (proces

Re: [PATCH] Limit removal of UBSAN_NULL checks

2014-11-05 Thread Jakub Jelinek
On Wed, Nov 05, 2014 at 10:37:46PM +0100, Marek Polacek wrote: > 2014-11-05 Marek Polacek > > * sanopt.c (sanopt_optimize_walker): Limit removal of the checks. > testsuite/ > * c-c++-common/ubsan/align-2.c: Add dg-output. > * c-c++-common/ubsan/align-4.c: Likewise. > * c

Re: [AArch64, Patch] Restructure arm_neon.h vector types's implementation(Take 2).

2014-11-05 Thread Marc Glisse
On Wed, 5 Nov 2014, Tejas Belagod wrote: 2014-10-01 Tejas Belagod * config/aarch64/aarch64-builtins.c (aarch64_build_scalar_type): Remove. (aarch64_scalar_builtin_types, aarch64_simd_type, aarch64_simd_type, aarch64_mangle_builtin_scalar_type, aar

[patch] Warn on undefined loop exit

2014-11-05 Thread Andrew Stubbs
This patch adds the following warning message: undefined.c:9:20: warning: statement may be undefined in the final loop iteration. [-Waggressive-loop-optimizations] for (i = 0; array[i] && i < 5; i++) ^ (Where the code ought to read "i < 5 && array[i]".) The tree-ssa loo

[PATCH] Limit removal of UBSAN_NULL checks

2014-11-05 Thread Marek Polacek
This patch limits optimizing UBSAN_NULL checks a bit, so that we don't lose diagnostics. If we know we're on a path where segv would have already occured, we can drop further checks (e.g. dereferencing a NULL pointer - we can't recover from that). Also if we're not recovering or using -fsanitize-u

Re: [PATCH] OpenACC for C++ front end

2014-11-05 Thread Joseph Myers
I think the TODO: XXX FIX -2 and TODO XXX: FIX -1 comments need, at least, more explanation of what the issue is and where the constants come from, even if something is left until later to be fixed. -- Joseph S. Myers jos...@codesourcery.com

[PATCH] OpenACC for C front end

2014-11-05 Thread James Norris
Hi! This patch represents the changes for OpenACC 2.0 in the C front-end. At present these files will not compile as the changes for the middle end are not present. OK to commit? Thanks, Jim => c/ChangeLog 2014-11-05 James Norris Cesar Philippidis Thomas S

[PATCH] OpenACC for C++ front end

2014-11-05 Thread James Norris
Hi! This patch represents the changes for OpenACC 2.0 in the C++ front-end. At present these files will not compile as the changes for the middle end are not present. Thanks, Jim => cp/ChangeLog 2014-11-05 James Norris Cesar Philippidis Thomas Schwinge

Re: [PATCH 00/13] Go closures, libffi, and the static chain

2014-11-05 Thread Lynn A. Boger
What about the libffi changes that are needed to make this work on other platforms, like PowerPC? On 10/10/2014 03:42 PM, Richard Henderson wrote: Pardon the wide distribution, the obvious hacks, and the failure to properly split up the largest of the libffi patches. The background here is my

Re: [gomp4] Use GOMP_OFFLOAD_ prefix for (OpenACC) plugin hooks

2014-11-05 Thread Julian Brown
On Wed, 5 Nov 2014 22:02:33 +0300 Ilya Verbin wrote: > Hi, > > On 05 Nov 17:56, Julian Brown wrote: > > +GOMP_OFFLOAD_register_image (void *host_table, void *target_data) > > +GOMP_OFFLOAD_get_table (struct mapping_table **table) > > FYI, these interfaces may change in the near future. > Curren

Re: Add uniform_inside_sphere_distribution

2014-11-05 Thread Marc Glisse
On Wed, 5 Nov 2014, Ed Smith-Rowland wrote: Like the uniform_on_sphere_distribution which is used inside, the 2-dimensional case uses rejection Could you point out where in the code you are special-casing dimension 2? Somehow I can't see it in the patch. -- Marc Glisse

Re: [PATCH] PR36312

2014-11-05 Thread Eric Botcazou
> 2014-10-25 Anthony Brandon > > PR driver/36312 > * diagnostic-core.h: Add prototype for fatal_error. > * diagnostic.c (fatal_error): New function fatal_error. > * gcc.c (store_arg): Remove have_o_argbuf_index. > (process_command): Check if input and output files are the sa

[PATCH 1/3] New test cases

2014-11-05 Thread David Malcolm
gcc/testsuite: * jit.dg/test-error-get-type-bad-enum.c: New. * jit.dg/test-error-new-binary-op-bad-op.c: New. * jit.dg/test-error-new-function-bad-kind.c: New. * jit.dg/test-error-new-unary-op-bad-op.c: New. --- .../jit.dg/test-error-get-type-bad-enum.c | 2

[PATCH 3/3] Add comments to various functions in libgccjit.h

2014-11-05 Thread David Malcolm
--- gcc/jit/libgccjit.h | 21 +++-- 1 file changed, 15 insertions(+), 6 deletions(-) diff --git a/gcc/jit/libgccjit.h b/gcc/jit/libgccjit.h index 8e03412..e07002d 100644 --- a/gcc/jit/libgccjit.h +++ b/gcc/jit/libgccjit.h @@ -123,11 +123,7 @@ typedef struct gcc_jit_lvalue gcc_jit_

[PATCH 2/3] Documentation tweak

2014-11-05 Thread David Malcolm
gcc/jit/ * docs/topics/expressions.rst (Type-coercion): Note that we support casting between pointer types. --- gcc/jit/docs/topics/expressions.rst | 1 + 1 file changed, 1 insertion(+) diff --git a/gcc/jit/docs/topics/expressions.rst b/gcc/jit/docs/topics/expressions.rst index a

[PATCH 0/3] Minor tweaks to jit

2014-11-05 Thread David Malcolm
Here are some minor tweaks I've made on the JIT branch since v3 of the review began. Are the following OK for trunk, assuming the rest of the JIT work is approved? David Malcolm (3): New test cases Documentation tweak Add comments to various functions in libgccjit.h gcc/jit/docs/topics/ex

Re: Extract and insert merging patch

2014-11-05 Thread Petr Murzin
Hi, Bootstrapped. No regressions detected. Please have a look. Is it ok for trunk? 2014-11-05 Petr Murzin gcc/ * simplify-rtx.c (simplify_ternary_operation): Simplify vec_merge (vec_duplicate (vec_select)). gcc/testsuite/ * gcc.target/i386/extract-insert-combining

Re: Re: [PATCH] Add missing requirement to crossmodule-indircall-1a.c

2014-11-05 Thread jb999
"Jeff Law" : >On 10/23/14 08:30, jb...@gmx.de wrote: >> "Jeff Law" : >> >>> On 10/21/14 12:21, jb...@gmx.de wrote: "Jeff Law" : > On 10/21/14 16:13, Haswell wrote: >> The additional source must have the same requirement >> crossmodule-indircall-1.c has. >> >> * crossmodul

[jit] Drop the disabled debugging code within handle_locations

2014-11-05 Thread David Malcolm
On Tue, 2014-11-04 at 15:21 -0700, Jeff Law wrote: > On 10/31/14 11:02, David Malcolm wrote: > > This files implements the gcc::jit::playback internal API, called by > > the dummy "frontend" to replay the public API calls made to the > > library. A thin wrapper around trees. > > > > gcc/jit/ > >

Re: The nvptx port [8/11+] Write undefined decls.

2014-11-05 Thread Jeff Law
On 11/05/14 05:01, Bernd Schmidt wrote: On 10/22/2014 08:11 PM, Jeff Law wrote: I'm not going to insist you do this in the same way as the PA. That was a different era -- we had significant motivation to make things work in such a way that everything could be buried in the pa specific files. Th

Re: [jit] Use ISALPHA and ISALNUM rather than writing our own

2014-11-05 Thread Jeff Law
On 11/05/14 08:48, David Malcolm wrote: On Tue, 2014-11-04 at 14:39 -0700, Jeff Law wrote: On 11/04/14 09:57, David Malcolm wrote: +#define IS_ASCII_DIGIT(CHAR) \ + ((CHAR) >= '0' && (CHAR) <='9') + +#define IS_ASCII_ALNUM(CHAR) \ + (IS_ASCII_ALPHA (CHAR) || IS_ASCII_DIGIT (CHAR)) Can't we r

Re: [PATCH 10/27] New file: gcc/jit/libgccjit.c

2014-11-05 Thread David Malcolm
On Tue, 2014-11-04 at 14:39 -0700, Jeff Law wrote: > On 11/04/14 09:57, David Malcolm wrote: > >>> +#define IS_ASCII_DIGIT(CHAR) \ > >>> + ((CHAR) >= '0' && (CHAR) <='9') > >>> + > >>> +#define IS_ASCII_ALNUM(CHAR) \ > >>> + (IS_ASCII_ALPHA (CHAR) || IS_ASCII_DIGIT (CHAR)) > >> Can't we rely on t

[jit] Add comments throughout libgccjit.c, and in libgccjit.h

2014-11-05 Thread David Malcolm
Committed to branch dmalcolm/jit: Also, add checking to ensure that gcc_jit_context_new_array_type fails with an error if given a negative size. gcc/jit/ChangeLog.jit: * docs/topics/expressions.rst (Type-coercion): Casts between pointer types are valid. * libgccjit.c: Docu

[Ada] Fix crash on function with In-Out param returning discriminated type

2014-11-05 Thread Eric Botcazou
The compiler crashes on a function with an In-Out parameter which returns a discriminated record type with default discriminant. Tested on x86_64-suse-linux, applied on the mainline. 2014-11-05 Eric Botcazou * gcc-interface/trans.c (Subprogram_Body_to_gnu): For a function with

[Ada] Fix ICE on type derived from private discriminated type

2014-11-05 Thread Eric Botcazou
The compiler aborts on a record type derived from a private discriminated record type without discriminant contraints, if the private discriminated record type is itself derived from another discriminated record type. Tested on x86_64-suse-linux, applied on the mainline. 2014-11-05 Eric Botca

Re: [gomp4] Use GOMP_OFFLOAD_ prefix for (OpenACC) plugin hooks

2014-11-05 Thread Ilya Verbin
Hi, On 05 Nov 17:56, Julian Brown wrote: > +GOMP_OFFLOAD_register_image (void *host_table, void *target_data) > +GOMP_OFFLOAD_get_table (struct mapping_table **table) FYI, these interfaces may change in the near future. Currently GOMP_OFFLOAD_get_table returns a joint table for all images, offloa

[Ada] Fix location information of exception block

2014-11-05 Thread Eric Botcazou
This makes it so that an exception block doesn't inherit a bogus location information in SJLJ mode. Tested on x86_64-suse-linux, applied on the mainline. 2014-11-05 Eric Botcazou * gcc-interface/trans.c (Handled_Sequence_Of_Statements_to_gnu): Set the SLOC of the node on the

Re: [Ada] Changes related to back-end inlining

2014-11-05 Thread Eric Botcazou
> 2014-10-31 Eric Botcazou > > * inline.adb (Has_Excluded_Declaration): With back-end inlining, > only return true for nested packages. > (Cannot_Inline): Issue errors/warnings whatever the optimization level > for back-end inlining and remove assertion. Here is a follo

Add uniform_inside_sphere_distribution

2014-11-05 Thread Ed Smith-Rowland
This distribution has come in handy for me. It relies on uniform_on_sphere_distribution and like it, min and max have no real meaning. Unlike uniform_on_sphere_distribution which really is a random multidimensional unit vector users often want to pick the radius of the distribution. Unit radiu

[jit] Verify enum values earlier

2014-11-05 Thread David Malcolm
It wasn't clear to me that all of these enum values were being fully validated by the internals, and it's better to fail early (so we can report which function was at fault), so explicitly validate enum values at the API entrypoints. The new testcases bring the # of expected passes in jit.sum from

[gomp4] Move libgomp plugins into subdirectory

2014-11-05 Thread Julian Brown
Hi, This patch moves plugin-nvptx.c and plugin-host.c (from oacc-host.c) into a new "plugin" subdirectory, as requested by Jakub, and to match more closely the layout of the Intel MIC pieces. This also moves the autotools bits to enable the NVPTX plugin and locate CUDA libraries into the plugin di

[gomp4] Use GOMP_OFFLOAD_ prefix for (OpenACC) plugin hooks

2014-11-05 Thread Julian Brown
Hi, Mirroring changes in Ilya Verbin's libgomp offloading pieces posted to trunk, this patch adds a prefix of GOMP_OFFLOAD_ to the OpenACC plugin hooks. Some of these bits will not be needed for a trunk version of the patch once Ilya's patch is approved (I'm hoping other incompatibilities haven't

Re: libstdc++ new deque failures

2014-11-05 Thread Jonathan Wakely
On 5 November 2014 14:14, David Edelsohn wrote: > Jonathan, > > I still am seeing new failures in the libstdc++ deque testsuite as of > last night. I don't know if you still are working through the fallout > from the earlier patches, but I wanted to make you aware. Yes, those tests are meant to f

Re: [PATCH] PR36312

2014-11-05 Thread Manuel López-Ibáñez
On 4 November 2014 23:40, Jeff Law wrote: > On 10/25/14 04:20, Anthony Brandon wrote: >> >> Hi, >> >> Sorry for the delay. Here are the updated diff and changelog. >> >> gcc/testsuite/ChangeLog: >> >> 2014-10-25 Anthony Brandon >> >> PR driver/36312 >> * gcc.misc-tests/output.exp: New

RE: [PATCHv3][MIPS] Implement O32 ABI extensions (GCC)

2014-11-05 Thread Matthew Fortune
Hi Catherine, The full patch is attached and the delta from v2 is inline below. Testing (O32): MIPS I - FP32, MIPS II - FP32, MIPS II - FPXX MIPS32 - FP32, MIPS32 - FPXX, MIPS32 - FPXX ODDSPREG, MIPS32R2 - FP32, MIPS32R2 - FPXX, MIPS32R2 - FPXX ODDSPREG, MIPS32R2 - FP64, MIPS32R2 - FP64A One of

[gomp4] OpenACC update host/self maintenance (was: acc enter/exit data)

2014-11-05 Thread Thomas Schwinge
Hi! On Thu, 30 Oct 2014 17:11:04 -0700, Cesar Philippidis wrote: > gcc/fortran/ > * gfortran.h (enum OMP_LIST_HOST): Remove. > (enum OMP_LIST_DEVICE, OMP_LIST_DEVICE): Remove. > * dump-parse-tree.c (show_omp_clauses): Remove OMP_LIST_HOST and > OMP_LIST_DEVICE from

[gomp4] Testing of C/C++ OpenACC cache directive (was: OpenACC cache directive for C)

2014-11-05 Thread Thomas Schwinge
Hi! On Wed, 05 Nov 2014 17:29:19 +0100, I wrote: > In r217145, I applied Jim's patch to gomp-4_0-branch: > * testsuite/libgomp.oacc-c/cache-1.c: New file. > * testsuite/libgomp.oacc-c++/cache-1.C: Likewise. Applied to gomp-4_0-branch in r217147: commit 267cffc4105255a8372fb5788fdcbb

[PATCH] Fix libbacktrace and libiberty tests fail on sanitized GCC due to wrong link options.

2014-11-05 Thread Maxim Ostapenko
Hi, When I ran Asan tests under Asan-bootstrapped GCC 5.0, I've noted, that tests for libiberty and libbacktrace fail to link with sanitized libbacktrace.a and libiberty.a because of missing -static-libasan -fsanitize=address linker flags. This patch adds necessary flags to provide a linkage o

Re: [gomp4] OpenACC cache directive maintenance

2014-11-05 Thread Thomas Schwinge
Hi Cesar! On Wed, 05 Nov 2014 17:36:46 +0100, I wrote: > In r217146, I applied the following to gomp-4_0-branch: > > commit e8e44b733808997d06c0cdf9bf5756ce03530f42 > Author: tschwinge > Date: Wed Nov 5 16:35:30 2014 + > > OpenACC cache directive maintenance. > > gcc/c/ >

[gomp4] OpenACC cache directive maintenance (was: [PATCH 4/6] [GOMP4] OpenACC 1.0+ support in fortran front-end)

2014-11-05 Thread Thomas Schwinge
Hi! On Fri, 24 Jan 2014 20:33:35 +0100, I wrote: > On Thu, 23 Jan 2014 22:04:45 +0400, Ilmir Usmanov > wrote: > > Subject: [PATCH 4/6] OpenACC GENERIC nodes > > --- a/gcc/tree-core.h > > +++ b/gcc/tree-core.h > > @@ -216,12 +216,18 @@ enum omp_clause_code { > > + /* Internal structure to hold

[PING 2] Enhance array types debug info. for Ada

2014-11-05 Thread Pierre-Marie de Rodat
Hello, Ping for https://gcc.gnu.org/ml/gcc-patches/2014-10/msg00694.html Thanks in advance! Regards, -- Pierre-Marie de Rodat

[gomp4] OpenACC cache directive for C.

2014-11-05 Thread Thomas Schwinge
Hi! In r217145, I applied Jim's patch to gomp-4_0-branch: commit 4361f9b6b2c74c2961c3a5290a4945abe2d7a444 Author: tschwinge Date: Wed Nov 5 16:26:47 2014 + OpenACC cache directive for C. gcc/c-family/ * c-pragma.c (oacc_pragmas): Add "cache". gcc/c/

Enable -fextended-identifiers by default

2014-11-05 Thread Joseph Myers
As proposed at , this patch enables -fextended-identifiers by default for all standard versions including this feature (all C++ versions, C99 and above for C, but not C90 / C94 / gnu89 / preprocessing assembler). It adds a couple of tests for areas

[gomp4] libgomp testsuite: OpenACC C++ testing (was: [2/3] OpenACC 2.0 support for libgomp - new tests)

2014-11-05 Thread Thomas Schwinge
Hi! Applied to gomp-4_0-branch in r217143: commit a78a06124f4047ec46a85e539e83640cc973aec1 Author: tschwinge Date: Wed Nov 5 16:16:14 2014 + libgomp testsuite: OpenACC C++ testing. libgomp/ * testsuite/libgomp.oacc-c++/c++.exp: Enable libgomp.oacc-c-c++-co

Re: [PATCH] Correctly check dg-require-effective-target in avx512 tests.

2014-11-05 Thread Uros Bizjak
On Wed, Nov 5, 2014 at 5:14 PM, Ilya Tocar wrote: > Hi, > > Currently we only check for dg-require-effective-target avx512vl in > avx512vl tests. We should also check for avx512dq/avx512bw. > Patch bwllow does this. > Ok for trunk? > > 2014-11-05 Ilya Tocar > > * gcc.target/i386/avx512v

[PATCH] Correctly check dg-require-effective-target in avx512 tests.

2014-11-05 Thread Ilya Tocar
Hi, Currently we only check for dg-require-effective-target avx512vl in avx512vl tests. We should also check for avx512dq/avx512bw. Patch bwllow does this. Ok for trunk? 2014-11-05 Ilya Tocar * gcc.target/i386/avx512vl-vandnpd-2.c: Fix dg-require-effective-target cehck.

[gomp4] OpenACC documentation updates.

2014-11-05 Thread Thomas Schwinge
Hi! Applied to gomp-4_0-branch in r217142: commit 0c5178ff5207bf1ede83070629c7d76fbbdf1afb Author: tschwinge Date: Wed Nov 5 16:12:51 2014 + OpenACC documentation updates. gcc/ * invoke.texi: Update for OpenACC. * sourcebuild.texi: Likewise. gcc/fo

Re: [gomp4] Rationalise thread-local variables in libgomp OpenACC support

2014-11-05 Thread Thomas Schwinge
Hi! On Wed, 29 Oct 2014 15:20:17 +, Julian Brown wrote: > On Tue, 28 Oct 2014 11:16:19 + > Julian Brown wrote: > > This patch rationalises TLS support by moving all thread-local > > variables into a single structure. Because this meant interfering with > > how per-thread/per-device init

Re: [debug-early] emit locals early patchset

2014-11-05 Thread Martin Jambor
On Tue, Oct 28, 2014 at 03:57:43PM +0100, Richard Biener wrote: > On Tue, Oct 28, 2014 at 1:00 AM, Aldy Hernandez wrote: > > Gentlemen! > > > > My apologies for the big patch. In getting locals emitted early (parameters > > and locally scoped variables), I ran into many things which were in need

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

2014-11-05 Thread Ilya Tocar
Ping. On 20 Oct 19:25, Ilya Tocar wrote: > Same in collect2. > > On 09 Oct 15:40, Ilya Tocar wrote: > > Ping. > > > > On 29 Sep 18:02, Ilya Tocar wrote: > > > Hi, > > > > > > Currently if call to atexit (lto_wrapper_cleanup) fails we > > > won't report error as we haven't initialized error-repo

[gomp4] libgomp OpenACC testsuite: Remove two obsolete test cases.

2014-11-05 Thread Thomas Schwinge
Hi! Applied to gomp-4_0-branch in r217140: commit 3838e13dedee9217b067cf2ab4b3fb5bb7d5cf68 Author: tschwinge Date: Wed Nov 5 16:02:30 2014 + libgomp OpenACC testsuite: Remove two obsolete test cases. libgomp/ * testsuite/libgomp.oacc-c/goacc_kernels.c: Remove file

Re: [gomp4] OpenACC / C++

2014-11-05 Thread Thomas Schwinge
Hi! On Wed, 15 Oct 2014 11:21:05 -0500, James Norris wrote: > This patch adds OpenACC support to C++ in the gomp4 branch. I found a few missing pieces; applied to gomp-4_0-branch in r217139: commit 09b8ef34550c377610c7a01aa2057fc8297e1b0a Author: tschwinge Date: Wed Nov 5 15:58:09 2014 +000

Re: [PATCH AVX512] Fix dg.torture tests with avx512

2014-11-05 Thread Ilya Tocar
On 03 Nov 11:21, Jakub Jelinek wrote: > On Fri, Oct 31, 2014 at 11:17:07AM +0100, Uros Bizjak wrote: > > I'd like to ask Jakub for a review of the above two parts, other parts > > are OK with a rename (as mentioned above). > > Looks ok to me. Where the ICEs discovered just by normal make check or

[gomp4] OpenACC Fortran testsuite: Expect some things to work by now.

2014-11-05 Thread Thomas Schwinge
Hi! Committed to gomp-4_0-branch in r217137: commit 83c3ae92fb16c23a782f012a49dd7aa1fcd01287 Author: tschwinge Date: Wed Nov 5 15:54:45 2014 + OpenACC Fortran testsuite: Expect some things to work by now. gcc/testsuite/ * gfortran.dg/goacc/data-tree.f95: Remove dg

Re: [gofrontend-dev] [PATCH 4/4] Gccgo port to s390[x] -- part II

2014-11-05 Thread Ian Taylor
On Wed, Nov 5, 2014 at 2:05 AM, Dominik Vogt wrote: > On Tue, Nov 04, 2014 at 08:16:51PM -0800, Ian Taylor wrote: >> I committed the change to go-test.exp. Thanks. >> >> The other changes are not OK. As described in >> gcc/testsuite/go.test/test/README.gcc, the files in >> gcc/testsuite/go.test/

[jit] Use ISALPHA and ISALNUM rather than writing our own

2014-11-05 Thread David Malcolm
On Tue, 2014-11-04 at 14:39 -0700, Jeff Law wrote: > On 11/04/14 09:57, David Malcolm wrote: > >>> +#define IS_ASCII_DIGIT(CHAR) \ > >>> + ((CHAR) >= '0' && (CHAR) <='9') > >>> + > >>> +#define IS_ASCII_ALNUM(CHAR) \ > >>> + (IS_ASCII_ALPHA (CHAR) || IS_ASCII_DIGIT (CHAR)) > >> Can't we rely on t

[gomp4] Remove unused OACC_WAIT (was: acc dealloc map)

2014-11-05 Thread Thomas Schwinge
Hi! On Mon, 20 Oct 2014 13:30:23 -0700, Cesar Philippidis wrote: > 2014-10-20 Cesar Philippidis > > gcc/ > * gimplify.c [...] > (gimplify_expr): Remove OACC_WAIT, since it handled directly by the > front ends. In r217136, I have now completely removed it: commit 1f7

[gomp4] Remove unused BUILT_IN_OMP_SET_NUM_THREADS (was: various OpenACC/PTX built-ins and a reduction tweak)

2014-11-05 Thread Thomas Schwinge
Hi! On Thu, 18 Sep 2014 20:43:20 +0200, I wrote: > On Tue, 16 Sep 2014 17:32:54 -0700, Cesar Philippidis > wrote: > > The patch [...] > > --- a/gcc/omp-builtins.def > > +++ b/gcc/omp-builtins.def > > @@ -236,6 +236,3 @@ DEF_GOMP_BUILTIN (BUILT_IN_GOMP_TARGET_UPDATE, > > "GOMP_target_update", >

Re: [patch,gomp-4_0-branch] acc nested function support

2014-11-05 Thread David Malcolm
On Tue, 2014-11-04 at 16:45 -0800, Cesar Philippidis wrote: > Here's an updated version of my nested function patch. > > David, I tweaked the gimple class hierarchy a little bit. Here's what > the updated class diagram looks like: > > + gimple_statement_omp > | |layout: GSS_OMP.

RE: [PATCH, i686] Fix for asan test failures with -m32 happened after EBX enabling in PIC mode

2014-11-05 Thread Zamyatin, Igor
> > Hi! > > > > Following patch (moving initialization of pic_offset_table_rtx > earlier) fixes failures for asan tests on 32 bits in PIC mode mentioned here - > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63534#c48 > > > > > > Bootstrapped/regtested on x86_64, i686 > > > > Is it ok for trunk? > >

Re: nvptx offloading patches [1/n]

2014-11-05 Thread Michael Matz
Hi, On Tue, 4 Nov 2014, Jeff Law wrote: > They still need to agree on the layout of the structure. And assuming > it'll always be memcpy perhaps isn't wise. Consider the possibility > that one day (perhaps soon) the host and GPU may share address space & > memory. Not only soon, there is al

Re: [PATCHv5][Kasan] Allow to override Asan shadow offset from command line

2014-11-05 Thread Kirill Yukhin
Hello, On 24 Oct 17:56, Yury Gribov wrote: ... > +const struct test_data_t test_data[] = { > + { STRTOL, "-0x8000", 0, -0x8000L, 0 }, ... > + switch (test_data[i].fun) > + { > + case STRTOL: > + res = strtol (test_data[i].nptr, 0, test_data[i].base); > + bre

Re: [PATCH, committed] Fix AIX testsuite failures

2014-11-05 Thread Rainer Orth
Hi David, > The appended patch XFAILs or adjusts testcases to avoid spurious > warnings on AIX. > > Bootstrapped on powerpc-ibm-aix7.1.0.0 > > Thanks, David > > * gcc.dg/torture/pr59166.c: XFAIL on AIX. > * g++.dg/ext/visitibility/anon1.C: XFAIL on AIX. > * g++.dg/opt/pr600

[PATCH, committed] Fix AIX testsuite failures

2014-11-05 Thread David Edelsohn
The appended patch XFAILs or adjusts testcases to avoid spurious warnings on AIX. Bootstrapped on powerpc-ibm-aix7.1.0.0 Thanks, David * gcc.dg/torture/pr59166.c: XFAIL on AIX. * g++.dg/ext/visitibility/anon1.C: XFAIL on AIX. * g++.dg/opt/pr60002.C: XFAIL on AIX.

libstdc++ new deque failures

2014-11-05 Thread David Edelsohn
Jonathan, I still am seeing new failures in the libstdc++ deque testsuite as of last night. I don't know if you still are working through the fallout from the earlier patches, but I wanted to make you aware. AIX defaults to 32 bit. A template was not initialized for int? FAIL: 23_containers/deq

[PATCH] PR 63721 IPA ICF cause atomic-comp-swap-release-acquire.c ICE

2014-11-05 Thread Jiong Wang
the same ICE will happen on x86-64, if compile with -O2 -fPIC. the reason is for the following two functions, they are identical, so IPA-ICF pass try to transform the second function to call the first one directly. int atomic_compare_exchange_STRONG_RELEASE_ACQUIRE (int a, int b) { return __

Re: [libcc1, build] Enable libcc1 on Solaris

2014-11-05 Thread Phil Muldoon
On 03/11/14 16:54, Rainer Orth wrote: > I noticed that the new libcc1 wasn't built on Solaris. This happens > because socketpair doesn't live in libc, but in libsocket instead. To > deal with this, I've copied the libgo (and libjava) code to detect the > need for libsocket and libnsl. Once the b

Re: [PATCH] Optimize UBSAN_NULL checks, add sanopt.c

2014-11-05 Thread Yury Gribov
On 11/05/2014 04:23 PM, Jakub Jelinek wrote: On Wed, Nov 05, 2014 at 04:13:01PM +0300, Yury Gribov wrote: Wouldn't it break most uses of __asan_poison_memory_region ? Most probably but I wonder if we should ask people to simply do asm volatile with memory clobber in this case? And we probably

Re: [Ping] [PATCH, 9/10] aarch64: generate conditional compare instructions

2014-11-05 Thread Richard Henderson
On 11/05/2014 10:05 AM, Zhenqiang Chen wrote: > I had retested all the ccmp patches. > > Bootstrap and no make check regression on X86-64. > Bootstrap and no make check regression on AARCH64 qemu. > > OK for trunk? No patch? Or what is it that you're wanting approval for? r~

Re: [PATCH] Optimize UBSAN_NULL checks, add sanopt.c

2014-11-05 Thread Jakub Jelinek
On Wed, Nov 05, 2014 at 04:13:01PM +0300, Yury Gribov wrote: > >>Wouldn't it break most uses of __asan_poison_memory_region ? > > > >Most probably but I wonder if we should ask people to simply do asm > >volatile with memory clobber in this case? And we probably shouldn't > >call the whole thing i

Re: [PATCH][AArch64][4.8] LINK_SPEC changes for Cortex-A53 erratum 835769 workaround

2014-11-05 Thread Marcus Shawcroft
On 22 October 2014 15:20, Kyrill Tkachov wrote: > Hi all, > > This is the 4.8 backport of the LINK_SPEC changes to pass down the linker > option > --fix-cortex-a53-835769 > > Bootstrapped and tested on aarch64-none-linux-gnu. > This depends on the patches under review at: > https://gcc.gnu.org/ml/

Re: [PATCH 3/n] OpenMP 4.0 offloading infrastructure: offload tables

2014-11-05 Thread Ilya Verbin
On 08 Oct 11:23, Jakub Jelinek wrote: > LGTM, with the requested var/section renames. > Would like if Honza and/or Richard had a look at the cgraph/LTO stuff > in the patch though. Since patch 2 was updated, this patch also should be updated. Now the offload_vars array is filled in varpool_node::g

Re: [Patch ARM-AArch64/testsuite v3 00/21] Neon intrinsics executable tests

2014-11-05 Thread Marcus Shawcroft
On 26 October 2014 16:50, Christophe Lyon wrote: > I've just realized afterwards that the tests aren't guarded against > targets not supporting Neon. > > How about adding the attached small patch? +if {[istarget arm*-*-*] +&& ![check_effective_target_arm_neon_ok]} then { + return +} + Umm,

Re: [Patch ARM-AArch64/testsuite v3 00/21] Neon intrinsics executable tests

2014-11-05 Thread Ramana Radhakrishnan
On Sun, Oct 26, 2014 at 4:50 PM, Christophe Lyon wrote: > On 24 October 2014 10:07, Marcus Shawcroft wrote: >> On 21 October 2014 14:02, Christophe Lyon wrote: >>> This patch series is an updated version of the series I sent here: >>> https://gcc.gnu.org/ml/gcc-patches/2014-07/msg00022.html >>>

Re: [PATCH] Optimize UBSAN_NULL checks, add sanopt.c

2014-11-05 Thread Yury Gribov
On 11/05/2014 03:34 PM, Yury Gribov wrote: On 11/05/2014 03:21 PM, Jakub Jelinek wrote: On Wed, Nov 05, 2014 at 03:16:49PM +0300, Yury Gribov wrote: On 11/05/2014 02:23 PM, Marek Polacek wrote: On Wed, Nov 05, 2014 at 11:50:20AM +0100, Jakub Jelinek wrote: On Wed, Nov 05, 2014 at 11:29:19AM +

Re: The nvptx port [10/11+] Target files

2014-11-05 Thread Bernd Schmidt
On 11/04/2014 05:51 PM, Bernd Schmidt wrote: On 11/04/2014 05:48 PM, Richard Henderson wrote: On 10/28/2014 03:56 PM, Bernd Schmidt wrote: +nvptx_ptx_type_from_mode (enum machine_mode mode, bool promote) +{ + switch (mode) +{ +case BLKmode: + return ".b8"; +case BImode: +

Re: [PARCH 1/2, x86, PR63534] Fix darwin bootstrap

2014-11-05 Thread Evgeny Stupachenko
We don't emit extra SET_GOT. That is beneficial. As for stack usage, that is RA to decide which register is more beneficial to put on stack. On Sat, Nov 1, 2014 at 8:33 PM, Mike Stump wrote: > On Nov 1, 2014, at 5:39 AM, Evgeny Stupachenko wrote: >> When PIC register is pseudo there is nothing s

Re: [PARCH 1/2, x86, PR63534] Fix darwin bootstrap

2014-11-05 Thread Evgeny Stupachenko
On Tue, Nov 4, 2014 at 1:40 AM, Jeff Law wrote: > On 11/01/14 06:39, Evgeny Stupachenko wrote: >> >> When PIC register is pseudo there is nothing special about it's value >> that setjmp can hurt. So if the pseudo register lives across >> setjmp_receiver RA should care about correct allocation (in

Re: [gofrontend-dev] [PATCH 1/4] Gccgo port to s390[x] -- part II

2014-11-05 Thread Dominik Vogt
On Wed, Nov 05, 2014 at 11:31:28AM +0100, Dominik Vogt wrote: > On Tue, Nov 04, 2014 at 02:39:34PM -0800, Ian Taylor wrote: > > Note that libgo/runtime/runtime.c now refers to S390_HAVE_STCKF. It's > > not obvious to me that that is defined anywhere. Perhaps it is in a > > later patch in this ser

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

2014-11-05 Thread Jakub Jelinek
On Wed, Nov 05, 2014 at 03:46:55PM +0300, Ilya Verbin wrote: > Maybe also with this change? > > diff --git a/gcc/omp-low.c b/gcc/omp-low.c > index 4e9ed25..beae5b5 100644 > --- a/gcc/omp-low.c > +++ b/gcc/omp-low.c > @@ -1653,8 +1653,7 @@ scan_sharing_clauses (tree clauses, omp_context *ctx) >

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

2014-11-05 Thread Ilya Verbin
On 03 Nov 10:24, Jakub Jelinek wrote: > On Tue, Oct 28, 2014 at 10:30:47PM +0300, Ilya Verbin wrote: > > @@ -474,6 +475,13 @@ cgraph_node::create (tree decl) > >gcc_assert (TREE_CODE (decl) == FUNCTION_DECL); > > > >node->decl = decl; > > + > > + if (lookup_attribute ("omp declare target

Re: [PATCH] Optimize UBSAN_NULL checks, add sanopt.c

2014-11-05 Thread Yury Gribov
On 11/05/2014 03:21 PM, Jakub Jelinek wrote: On Wed, Nov 05, 2014 at 03:16:49PM +0300, Yury Gribov wrote: On 11/05/2014 02:23 PM, Marek Polacek wrote: On Wed, Nov 05, 2014 at 11:50:20AM +0100, Jakub Jelinek wrote: On Wed, Nov 05, 2014 at 11:29:19AM +0100, Marek Polacek wrote: On Wed, Nov 05,

  1   2   >