[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: 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

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

[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

[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 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 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

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

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: [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: [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

[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

[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

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] 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

[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

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

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: [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 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: [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 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: [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: [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] 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: 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

[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: [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

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

[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

[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

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

[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 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

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

[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

<    1   2