[PATCH, testsuite] Add effective target stack_size

2017-06-09 Thread Tom de Vries
sofar. OK for trunk if x86_64 and nvptx testing succeeds? Thanks, - Tom Add effective target stack_size 2017-06-09 Tom de Vries * lib/target-supports.exp (check_effective_target_stack_size) (dg-effective-target-value): New proc. * lib/gcc.exp (gcc_target_compile): Remove adding of DSTACK_SIZ

[nvptx, committed, PR80855] Add "sorry, target cannot support label values" for nvptx

2017-06-09 Thread Tom de Vries
rt label values" for nvptx 2017-06-09 Tom de Vries PR target/80855 * config/nvptx/nvptx.md (define_expand "mov"): Error out with "target cannot support label values" when encountering LABEL_REF. * gcc.target/nvptx/label-values.c: New test. --- gcc/config/nvptx/nv

[RFC] Dejagnu patch to handle multi-line directives

2017-06-10 Thread Tom de Vries
Hi, one thing that has bothered me on a regular basis is the inability to spread long dejagnu directives over multiple lines. I've written a demonstrator patch (for the dejagnu sources) and tested it by splitting this 108 chars line: ... /* { dg-additional-options "-DSTACK_SIZE=[dg-effective

Re: [RFC] Dejagnu patch to handle multi-line directives

2017-06-10 Thread Tom de Vries
[ attached patch ] On 06/10/2017 09:57 AM, Tom de Vries wrote: Hi, one thing that has bothered me on a regular basis is the inability to spread long dejagnu directives over multiple lines. I've written a demonstrator patch (for the dejagnu sources) and tested it by splitting this 108

[nvptx, committed, PR79939] Disable constant pool for nvptx

2017-06-11 Thread Tom de Vries
Hi, this patch fixes PR79939, a cc1 hang while trying to emit a constant in the constant pool. Fixed by disabling the constant pool for nvptx. Tested on target nvptx. Committed as obvious. Thanks, - Tom Disable constant pool for nvptx 2017-06-11 Tom de Vries PR target/79939 * config

Re: [PATCH, testsuite] Add effective target stack_size

2017-06-12 Thread Tom de Vries
On 06/12/2017 02:28 PM, Christophe Lyon wrote: Hi Tom, On 9 June 2017 at 17:25, Mike Stump wrote: On Jun 9, 2017, at 7:24 AM, Tom de Vries wrote: this patch adds effective target stack_size. OK for trunk if x86_64 and nvptx testing succeeds? Ok. The only last issue in this area that I

Add dg-add-options feature stack_size

2017-06-19 Thread Tom de Vries
[ was: Re: [PATCH, testsuite] Add effective target stack_size ] On 06/09/2017 05:25 PM, Mike Stump wrote: On Jun 9, 2017, at 7:24 AM, Tom de Vries wrote: this patch adds effective target stack_size. OK for trunk if x86_64 and nvptx testing succeeds? Ok. Hi, I came across dg-add-options

[testsuite, committed] Fix warning in gcc.dg/tree-prof/comp-goto-1.c

2017-06-21 Thread Tom de Vries
Hi, this patch fixes a 'return type defaults to int' warning in test-case gcc.dg/tree-prof/comp-goto-1.c. Committed as obvious. Thanks, - Tom Fix warning in gcc.dg/tree-prof/comp-goto-1.c 2017-06-21 Tom de Vries * gcc.dg/tree-prof/comp-goto-1.c: Fix 'return type

[testsuite, committed] Support dg-add-options in profopt.exp

2017-06-21 Thread Tom de Vries
[ was: Re: [PATCH, testsuite] Add effective target stack_size ] On 06/21/2017 03:19 PM, Jakub Jelinek wrote: On Fri, Jun 09, 2017 at 04:24:30PM +0200, Tom de Vries wrote: * gcc.dg/tree-prof/comp-goto-1.c: Same. * gcc.dg/tree-prof/pr44777.c: Same. --- a/gcc/testsuite/gcc.dg

[PATCH, 0/4] Handle GOMP_OPENACC_NVPTX_{DISASM,SAVE_TEMPS,JIT} in libgomp nvptx plugin

2017-06-26 Thread Tom de Vries
Hi, I've written a patch series to facilitate debugging libgomp openacc testcase failures on the nvptx accelerator. When running an openacc test-case on an nvptx accelerator, the following happens: - the plugin obtains the ptx assembly for the acceleration kernels - it calls the cuda jit to

[PATCH, 1/4] Show value of GOMP_OPENACC_DIM in libgomp nvptx plugin

2017-06-26 Thread Tom de Vries
On 06/26/2017 01:24 PM, Tom de Vries wrote: Hi, I've written a patch series to facilitate debugging libgomp openacc testcase failures on the nvptx accelerator. When running an openacc test-case on an nvptx accelerator, the following happens: - the plugin obtains the ptx assembly fo

[PATCH, 2/4] Handle GOMP_OPENACC_NVPTX_{DISASM,SAVE_TEMPS} in libgomp nvptx plugin

2017-06-26 Thread Tom de Vries
On 06/26/2017 01:24 PM, Tom de Vries wrote: Hi, I've written a patch series to facilitate debugging libgomp openacc testcase failures on the nvptx accelerator. When running an openacc test-case on an nvptx accelerator, the following happens: - the plugin obtains the ptx assembly fo

Re: [PATCH, 0/4] Handle GOMP_OPENACC_NVPTX_{DISASM,SAVE_TEMPS,JIT} in libgomp nvptx plugin

2017-06-26 Thread Tom de Vries
On 06/26/2017 01:24 PM, Tom de Vries wrote: Hi, I've written a patch series to facilitate debugging libgomp openacc testcase failures on the nvptx accelerator. When running an openacc test-case on an nvptx accelerator, the following happens: - the plugin obtains the ptx assembly fo

[PATCH, 4/4] Handle GOMP_OPENACC_NVPTX_JIT=-ori in libgomp nvptx plugin

2017-06-26 Thread Tom de Vries
On 06/26/2017 01:24 PM, Tom de Vries wrote: Hi, I've written a patch series to facilitate debugging libgomp openacc testcase failures on the nvptx accelerator. When running an openacc test-case on an nvptx accelerator, the following happens: - the plugin obtains the ptx assembly fo

[PATCH, 3/4] Handle GOMP_OPENACC_NVPTX_JIT=-O[0-4] in libgomp nvptx plugin

2017-06-26 Thread Tom de Vries
[ reposting with proper subject ] On 06/26/2017 01:42 PM, Tom de Vries wrote: On 06/26/2017 01:24 PM, Tom de Vries wrote: Hi, I've written a patch series to facilitate debugging libgomp openacc testcase failures on the nvptx accelerator. When running an openacc test-case on an

[PATCH] Use secure_getenv for GOMP_DEBUG

2017-06-27 Thread Tom de Vries
[ was: Re: [PATCH, 2/4] Handle GOMP_OPENACC_NVPTX_{DISASM,SAVE_TEMPS} in libgomp nvptx plugin ] On 06/26/2017 05:29 PM, Jakub Jelinek wrote: On Mon, Jun 26, 2017 at 03:26:57PM +, Joseph Myers wrote: On Mon, 26 Jun 2017, Tom de Vries wrote: 2. Handle GOMP_OPENACC_NVPTX_{DISASM,SAVE_TEMPS

[PATCH, 5/4] Handle GOMP_OPENACC_NVPTX_PTXRW in libgomp nvptx plugin

2017-06-27 Thread Tom de Vries
[ was: Re: [PATCH, 0/4] Handle GOMP_OPENACC_NVPTX_{DISASM,SAVE_TEMPS,JIT} in libgomp nvptx plugin ] On 06/26/2017 01:24 PM, Tom de Vries wrote: Hi, I've written a patch series to facilitate debugging libgomp openacc testcase failures on the nvptx accelerator. When running an openacc

Re: [PATCH] Use secure_getenv for GOMP_DEBUG

2017-06-27 Thread Tom de Vries
On 06/27/2017 09:38 AM, Jakub Jelinek wrote: On Tue, Jun 27, 2017 at 09:17:57AM +0200, Tom de Vries wrote: This patch uses secure_getenv for GOMP_DEBUG. It factors out the secure_getenv code from plugin-hsa.c into libgomp/secure_getenv.h, and reuses it in env.c. I've added _GNU_SOURCE b

Re: [PATCH, 2/4] Handle GOMP_OPENACC_NVPTX_{DISASM,SAVE_TEMPS} in libgomp nvptx plugin

2017-06-27 Thread Tom de Vries
On 06/26/2017 05:29 PM, Jakub Jelinek wrote: On Mon, Jun 26, 2017 at 03:26:57PM +, Joseph Myers wrote: On Mon, 26 Jun 2017, Tom de Vries wrote: 2. Handle GOMP_OPENACC_NVPTX_{DISASM,SAVE_TEMPS} in libgomp nvptx plugin This patch adds handling of: - GOMP_OPENACC_NVPTX_SAVE_TEMPS=[01], and

Re: [PATCH, 1/4] Show value of GOMP_OPENACC_DIM in libgomp nvptx plugin

2017-06-27 Thread Tom de Vries
On 06/26/2017 01:31 PM, Tom de Vries wrote: On 06/26/2017 01:24 PM, Tom de Vries wrote: Hi, I've written a patch series to facilitate debugging libgomp openacc testcase failures on the nvptx accelerator. When running an openacc test-case on an nvptx accelerator, the following ha

[gomp4, nvptx, committed] Fix assert in nvptx_propagate_unified

2017-06-30 Thread Tom de Vries
e included in the patch. Committed as trivial. Thanks, - Tom Fix assert in nvptx_propagate_unified 2017-06-30 Tom de Vries * config/nvptx/nvptx.c (nvptx_propagate_unified): Fix gcc_assert condition by allowing !INSN_P. * testsuite/libgomp.oacc-c-c++-common/reduction-cplx-flt

Re: [PATCH, 4/4] Handle GOMP_OPENACC_NVPTX_JIT=-ori in libgomp nvptx plugin

2017-06-30 Thread Tom de Vries
On 06/26/2017 01:44 PM, Tom de Vries wrote: On 06/26/2017 01:24 PM, Tom de Vries wrote: Hi, I've written a patch series to facilitate debugging libgomp openacc testcase failures on the nvptx accelerator. When running an openacc test-case on an nvptx accelerator, the following ha

Re: [PATCH, 5/4] Handle GOMP_OPENACC_NVPTX_PTXRW in libgomp nvptx plugin

2017-06-30 Thread Tom de Vries
On 06/27/2017 11:16 AM, Tom de Vries wrote: [ was: Re: [PATCH, 0/4] Handle GOMP_OPENACC_NVPTX_{DISASM,SAVE_TEMPS,JIT} in libgomp nvptx plugin ] On 06/26/2017 01:24 PM, Tom de Vries wrote: Hi, I've written a patch series to facilitate debugging libgomp openacc testcase failures on the

[PATCH, 6/4] Handle GOMP_OPENACC_NVPTX_JIT=-arch= in libgomp nvptx plugin

2017-06-30 Thread Tom de Vries
[ was: Re: [PATCH, 0/4] Handle GOMP_OPENACC_NVPTX_{DISASM,SAVE_TEMPS,JIT} in libgomp nvptx plugin ] On 06/26/2017 01:24 PM, Tom de Vries wrote: Hi, I've written a patch series to facilitate debugging libgomp openacc testcase failures on the nvptx accelerator. When running an openacc

[PATCH, PR81192] Fix sigsegv in find_same_succ_bb

2017-07-02 Thread Tom de Vries
the tail-merge pass. This patch fixes the sigsegv by doing an early-out in same_succ_flush_bb if BB_SAME_SUCC () == NULL. Bootstrapped and reg-tested on x86_64. OK for trunk and gcc-[567]-branch? Thanks, - Tom Fix sigsegv in find_same_succ_bb 2017-06-30 Tom de Vries PR tree-optimizat

[PATCH, PR81192] Don't tail-merge blocks from different loops

2017-07-02 Thread Tom de Vries
[ was: Re: [PATCH, PR81192] Fix sigsegv in find_same_succ_bb ] On 07/03/2017 12:26 AM, Tom de Vries wrote: [ Trying again with before.svg instead of before.pdf ] Hi, consider this test-case: ... unsigned a; int b, c; static int fn1 (int p1, int p2) { return p1 > 2147483647 - p2 ? p1 :

[PATCH, PR69468] Ignore EDGE_{DFS_BACK,EXECUTABLE} in tail-merge

2017-07-02 Thread Tom de Vries
[ was: Re: [PATCH, PR81192] Don't tail-merge blocks from different loops ] On 07/03/2017 12:49 AM, Tom de Vries wrote: [ was: Re: [PATCH, PR81192] Fix sigsegv in find_same_succ_bb ] On 07/03/2017 12:26 AM, Tom de Vries wrote: [ Trying again with before.svg instead of before.pdf

[PATCH] Add dotfn

2017-07-02 Thread Tom de Vries
. Bootstrapped and reg-tested on x86_64. Used for debugging PR81192. OK for trunk? Thanks, - Tom Add dotfn 2017-06-30 Tom de Vries * graph.c (get_graph_file_name): New function, factored out of ... (open_graph_file): ... here. (dotfn): New function. * graph.h (dotfn): Declare. (DOTFN): New macro

Re: [PATCH] Add dotfn

2017-07-03 Thread Tom de Vries
On 07/03/2017 09:05 AM, Richard Biener wrote: On Mon, 3 Jul 2017, Tom de Vries wrote: Hi, this patch adds a debug function dotfn and a convenience macro DOTFN similar to dot-fn in gdbhooks.py. It can be used to have the compiler: - dump a control flow graph, or - pop up a control flow graph

Re: [PATCH] Add dotfn

2017-07-03 Thread Tom de Vries
On 07/03/2017 11:53 AM, Richard Biener wrote: On Mon, 3 Jul 2017, Tom de Vries wrote: On 07/03/2017 09:05 AM, Richard Biener wrote: On Mon, 3 Jul 2017, Tom de Vries wrote: Hi, this patch adds a debug function dotfn and a convenience macro DOTFN similar to dot-fn in gdbhooks.py. It can be

Re: [PATCH] Use secure_getenv for GOMP_DEBUG

2017-07-03 Thread Tom de Vries
On 07/03/2017 02:26 PM, Franz Sirl wrote: Am 27.06.17 um 13:10 schrieb Tom de Vries: --- a/libgomp/plugin/plugin-hsa.c +++ b/libgomp/plugin/plugin-hsa.c @@ -39,32 +39,7 @@ #include #include "libgomp-plugin.h" #include "gomp-constants.h" - -/* Secure getenv() which ret

Re: [PATCH, 2/4] Handle GOMP_OPENACC_NVPTX_{DISASM,SAVE_TEMPS} in libgomp nvptx plugin

2017-07-03 Thread Tom de Vries
On 07/03/2017 04:08 PM, Thomas Schwinge wrote: Hi! On Mon, 26 Jun 2017 17:29:11 +0200, Jakub Jelinek wrote: On Mon, Jun 26, 2017 at 03:26:57PM +, Joseph Myers wrote: On Mon, 26 Jun 2017, Tom de Vries wrote: 2. Handle GOMP_OPENACC_NVPTX_{DISASM,SAVE_TEMPS} in libgomp nvptx plugin This

[PATCH] Save and restore EDGE_DFS_BACK in draw_cfg_edges

2017-07-03 Thread Tom de Vries
Adding this patch makes the sigsegv come back. OK for trunk if bootstrap and reg-test on x86_64 succeeds? Thanks, - Tom Save and restore EDGE_DFS_BACK in draw_cfg_edges 2017-07-03 Tom de Vries * graph.c (draw_cfg_edges): Save and restore EDGE_DFS_BACK. ---

Re: [PATCH, 2/4] Handle GOMP_OPENACC_NVPTX_{DISASM,SAVE_TEMPS} in libgomp nvptx plugin

2017-07-04 Thread Tom de Vries
On 07/03/2017 04:24 PM, Tom de Vries wrote: On 07/03/2017 04:08 PM, Thomas Schwinge wrote: Hi! On Mon, 26 Jun 2017 17:29:11 +0200, Jakub Jelinek wrote: On Mon, Jun 26, 2017 at 03:26:57PM +, Joseph Myers wrote: On Mon, 26 Jun 2017, Tom de Vries wrote: 2. Handle GOMP_OPENACC_NVPTX_

[PATCH, 1/3] Handle GOMP_NVPTX_{DISASM,SAVE_TEMPS} in libgomp nvptx plugin

2017-07-04 Thread Tom de Vries
On 07/04/2017 12:05 PM, Tom de Vries wrote: On 07/03/2017 04:24 PM, Tom de Vries wrote: On 07/03/2017 04:08 PM, Thomas Schwinge wrote: Hi! On Mon, 26 Jun 2017 17:29:11 +0200, Jakub Jelinek wrote: On Mon, Jun 26, 2017 at 03:26:57PM +, Joseph Myers wrote: On Mon, 26 Jun 2017, Tom de

[PATCH, 2/3] Handle GOMP_NVPTX_PTXRW in libgomp nvptx plugin

2017-07-04 Thread Tom de Vries
On 07/04/2017 12:05 PM, Tom de Vries wrote: On 07/03/2017 04:24 PM, Tom de Vries wrote: On 07/03/2017 04:08 PM, Thomas Schwinge wrote: Hi! On Mon, 26 Jun 2017 17:29:11 +0200, Jakub Jelinek wrote: On Mon, Jun 26, 2017 at 03:26:57PM +, Joseph Myers wrote: On Mon, 26 Jun 2017, Tom de

[PATCH, 3/3] Handle GOMP_NVPTX_JIT={-O[0-4],-ori,-arch=} in libgomp nvptx plugin

2017-07-04 Thread Tom de Vries
On 07/04/2017 12:05 PM, Tom de Vries wrote: On 07/03/2017 04:24 PM, Tom de Vries wrote: On 07/03/2017 04:08 PM, Thomas Schwinge wrote: Hi! On Mon, 26 Jun 2017 17:29:11 +0200, Jakub Jelinek wrote: On Mon, Jun 26, 2017 at 03:26:57PM +, Joseph Myers wrote: On Mon, 26 Jun 2017, Tom de

[RFC] Add -fmap-abort-to-trap

2017-07-04 Thread Tom de Vries
l usefulness to this patch, but I'm posting it here in case anybody finds it useful. Thanks, - Tom Add -fmap-abort-to-trap 2017-06-21 Tom de Vries * builtins.c (expand_builtin): Map abort and __builtin_abort to __builtin_trap for -fmap-abort-to-trap. * common.opt (fmap-abort-to-tra

[mips, committed] Remove semicolon after do {} while (0) in ASM_OUTPUT_CASE_END

2017-11-08 Thread Tom de Vries
_END 2017-11-07 Tom de Vries * config/mips/mips.h (ASM_OUTPUT_CASE_END): Remove semicolon after "do {} while (0)". --- gcc/config/mips/mips.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gcc/config/mips/mips.h b/gcc/config/mips/mips.h index d7f86b6..c1fc

[mips, committed] Wrap ASM_OUTPUT_LABELREF in do {} while (0)

2017-11-08 Thread Tom de Vries
if (d) ^ ... Build for mips target. Committed as obvious. Thanks, - Tom [mips] Wrap ASM_OUTPUT_LABELREF in do {} while (0) 2017-11-07 Tom de Vries * config/mips/mips.h (ASM_OUTPUT_LABELREF): Wrap in "do {} while (0)". --- gcc/config/mips/mips.h | 15 +-- 1 file changed

Re: [PATCH 1/7] [libsanitizer, committed] Remove semicolon after do {} while (0) in macro body

2017-11-08 Thread Tom de Vries
On 11/08/2017 09:16 AM, Jakub Jelinek wrote: On Sun, Nov 05, 2017 at 11:02:32AM +0100, Tom de Vries wrote: this removes a semicolon after "do {} while (0)" in several macro bodies in libsanitizer. This allows the macros to be used in if-then-elses without curly braces. Committed

[cr16, powerpcspe, rs6000] Remove semicolon after ASM_OUTPUT_LABELREF macro body

2017-11-12 Thread Tom de Vries
Hi, this removes a semicolon after the ASM_OUTPUT_LABELREF macro body. Committed as obvious. Thanks, - Tom [cr16, powerpcspe, rs6000] Remove semicolon after ASM_OUTPUT_LABELREF macro body 2017-11-12 Tom de Vries * config/cr16/cr16.h (ASM_OUTPUT_LABELREF): Remove semicolon after macro

[committed] Remove semicolon after ASM_OUTPUT_ASCII

2017-11-12 Thread Tom de Vries
Hi, this patch removes a semicolon after the ASM_OUTPUT_ASCII macro body in elfos.h. Build for x86_64. Committed as obvious. Thanks, - Tom Remove semicolon after ASM_OUTPUT_ASCII 2017-11-12 Tom de Vries * config/elfos.h (ASM_OUTPUT_ASCII): Remove semicolon after macro body. --- gcc

[riscv] Wrap ASM_OUTPUT_LABELREF in do {} while (0)

2017-11-12 Thread Tom de Vries
Hi, this patch wraps riscv.h's ASM_OUTPUT_LABELREF in "do {} while (0)". Build for riscv64. Committed as obvious. Thanks, - Tom [riscv] Wrap ASM_OUTPUT_LABELREF in do {} while (0) 2017-11-12 Tom de Vries * config/riscv/riscv.h (ASM_OUTPUT_LABELREF): Wrap in do {} while

[testsuite, committed] Require alloca for c-c++-common/Wstringop-truncation.c

2017-11-14 Thread Tom de Vries
) +{ + char vla[n]; Hi, this patch requires effective target alloca for test-case c-c++-common/Wstringop-truncation.c. Committed as obvious. Thanks, - Tom Require alloca for c-c++-common/Wstringop-truncation.c 2017-11-14 Tom de Vries * c-c++-common/Wstringop-truncation.c: Require effective

[libgomp, testsuite, committed] Allow asyncwait-1.c to run for non-nvidia devices

2017-11-14 Thread Tom de Vries
Hi, this patch allows openacc test-case asyncwait-1.c to run for non-nvidia devices. Committed as obvious. Thanks, - Tom Allow asyncwait-1.c to run for non-nvidia devices 2017-11-14 Tom de Vries * testsuite/libgomp.oacc-c-c++-common/asyncwait-1.c: Allow to run for non-nvidia devices

Add libgomp.oacc-c-c++-common/f-asyncwait-{1,2,3}.c

2017-11-15 Thread Tom de Vries
itted as obvious. Thanks, - Tom Add libgomp.oacc-c-c++-common/f-asyncwait-{1,2,3}.c 2017-11-15 Tom de Vries * testsuite/libgomp.oacc-c-c++-common/f-asyncwait-1.c: New test, copied from asyncwait-1.f90. Rewrite into C. Rewrite from float to int. * testsuite/libgomp.oacc-c-c++-common/

[testsuite, committed] Compile strncpy-fix-1.c with -Wno-stringop-truncation

2017-11-15 Thread Tom de Vries
ation] ... Fixed by adding -Wno-stringop-truncation. Committed as obvious. Thanks, - Tom Compile strncpy-fix-1.c with -Wno-stringop-truncation 2017-11-15 Tom de Vries * gcc.dg/strncpy-fix-1.c: Add -Wno-stringop-truncation to dg-options. --- gcc/testsuite/gcc.dg/strncpy-fix-1.c | 2 +- 1

[PATCH, libgomp, openacc] Use GOMP_ASYNC_SYNC in GOACC_declare

2017-11-17 Thread Tom de Vries
Hi, GOACC_enter_exit_data has this prototype: ... void GOACC_enter_exit_data (int device, size_t mapnum, void **hostaddrs, size_t *sizes, unsigned short *kinds, int async, int num_waits, ...) ... And GOACC_declare calls GOACC_e

RFC: libgomp target plugins and atexit

2017-11-17 Thread Tom de Vries
Hi, I wrote a patch that called some function in the common libgomp code from GOMP_OFFLOAD_fini_device, and found that it hung due to the fact that: - gomp_target_fini locks devices[*].lock while calling GOMP_OFFLOAD_fini_device, and - the function call that I added also locked that same lock

[PATCH, libgomp, openacc] Factor out async argument utility functions

2017-11-17 Thread Tom de Vries
in values of acc_async_noval and acc_async_sync. No functional changes. Build and tested on x86_64 with nvptx accelerator. OK for trunk if bootstrap and reg-test on x86_64 succeeds? Thanks, - Tom Factor out async argument utility functions 2017-11-17 Tom de Vries * oacc-int.h (async_vali

[vms, committed] Add missing vmsdbgout_early_finish

2017-11-18 Thread Tom de Vries
it in will probably unbreak the build). Thanks, - Tom [vms] Add missing vmsdbgout_early_finish 2017-11-18 Tom de Vries PR target/82961 * vmsdbgout.c (vmsdbgout_early_finish): New function. (vmsdbg_debug_hooks): Set early_finish field to vmsdbgout_early_finish. --- gcc/vmsdbgout.c | 1

Re: Increase precision of static profiles

2017-11-19 Thread Tom de Vries
On 11/17/2017 08:53 PM, Jan Hubicka wrote: Hi, this patch makes static profile to be in range 0...2^30 rather than 0...1. This is safe now as profile-counts are taking care of possible overflow when the profile ends up cummulating too high after inlining. MThere are two testcases that needs

[mcore, committed] Remove semicolon after do {} while (0) in MCORE_EXPORT_NAME

2017-11-19 Thread Tom de Vries
NAME 2017-11-14 Tom de Vries * config/mcore/mcore-elf.h (MCORE_EXPORT_NAME): Remove semicolon after "do {} while (0)". * config/mcore/mcore.h (ASM_OUTPUT_ALIGNED_COMMON): After missing semicolon after MCORE_EXPORT_NAME call. --- gcc/config/mcore/mcore-elf.h | 2 +- gcc/config/m

[ft32, spu, committed] Remove semicolon after do {} while (0) in REGISTER_TARGET_PRAGMAS

2017-11-19 Thread Tom de Vries
ile (0) in REGISTER_TARGET_PRAGMAS 2017-11-14 Tom de Vries * config/ft32/ft32.h (REGISTER_TARGET_PRAGMAS): Remove semicolon after "do {} while (0)". * config/spu/spu.h (REGISTER_TARGET_PRAGMAS): Same. --- gcc/config/ft32/ft32.h | 2 +- gcc/config/spu/spu.h | 2 +- 2 files changed, 2 insert

[visium, committed] Remove semicolon after ASM_OUTPUT_CASE_END

2017-11-19 Thread Tom de Vries
Hi, this patch removes a semicolon after ASM_OUTPUT_CASE_END. This allows the macro to be used in if-then-elses without curly braces. Build visium-unknown-elf. Committed as obvious. Thanks, - Tom [visium] Remove semicolon after ASM_OUTPUT_CASE_END 2017-11-14 Tom de Vries * config

[phoenix, committed] Remove semicolon after do {} while (0) in TARGET_OS_CPP_BUILTINS

2017-11-19 Thread Tom de Vries
ile (0) in TARGET_OS_CPP_BUILTINS 2017-11-14 Tom de Vries * config/phoenix.h (TARGET_OS_CPP_BUILTINS): Remove semicolon after "do {} while (0)". --- gcc/config/phoenix.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gcc/config/phoenix.h b/gcc/config/phoenix.h index 64728a5..

[libgcc, alpha, vms] Remove semicolon after do {} while (0) in UPDATE_FS_FOR_CFA_GR

2017-11-19 Thread Tom de Vries
this patch, or approve as trivial? Thanks, - Tom [libgcc, alpha, vms] Remove semicolon after do {} while (0) in UPDATE_FS_FOR_CFA_GR 2017-11-19 Tom de Vries * config/alpha/vms-unwind.h (UPDATE_FS_FOR_CFA_GR): Remove semicolon after "do {} while (0)". --- libgcc/config/alpha/vm

[arc] Remove semicolon after do while (0) in FUNCTION_PROFILER

2017-11-19 Thread Tom de Vries
olon after do while (0) in FUNCTION_PROFILER 2017-11-19 Tom de Vries * config/arc/arc.h (FUNCTION_PROFILER): Remove semicolon after "do while (0)". --- gcc/config/arc/arc.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gcc/config/arc/arc.h b/gcc/config/arc

Re: [PATCH] Fix bug in simplify_ternary_operation

2017-11-20 Thread Tom de Vries
On 11/20/2017 03:52 AM, Jeff Law wrote: Sorry, it's taken so long to get back to this patch... On 09/01/2017 02:51 AM, Tom de Vries wrote: On 08/31/2017 11:44 PM, Jeff Law wrote: On 08/28/2017 12:26 PM, Tom de Vries wrote: Hi, I think I found a bug in r17465: ...     *

RFC: SAFE_MACRO_STMT

2017-11-20 Thread Tom de Vries
Hi, when writing macros that are intended to be used as if they were single statements, there are a couple of common problems: 1. Ending in semicolon ... #define foo stmt; ... This works fine if we use it like: ... void bar (void) { foo; } ... but we get an "else without a previous if" err

[PATCH] Handle GOMP_NVPTX_PTXRW in libgomp nvptx plugin

2017-11-21 Thread Tom de Vries
it doesn't. OK for trunk if x86_64 bootstrap and reg-test succeeds? Thanks, - Tom Handle GOMP_NVPTX_PTXRW in libgomp nvptx plugin 2017-11-21 Tom de Vries * plugin/plugin-nvptx.c (_GNU_SOURCE): Define. (gomp_nvptx_ptxrw): New static variable. (parse_gomp_nvptx_ptxr

Re: [SH][committed] Cleanup div and mul patterns

2017-11-22 Thread Tom de Vries
On 05/03/2016 08:50 AM, Oleg Endo wrote: Hi, The attached patch cleans up the SH div and mul patterns a bit. Almost no changes in CSiBE, except for a few minor improvements. Tested on sh-elf with make -k check RUNTESTFLAGS="--target_board=sh-sim\{-m2/-ml,-m2/-mb, -m2a/-mb,-m4/-ml,-m4/-mb,-m4a/

[ft32, committed] Remove semicolon after ASM_OUTPUT_ADDR_VEC_ELT

2017-11-23 Thread Tom de Vries
Hi, This removes a semicolon after the ft32 version of ASM_OUTPUT_ADDR_VEC_ELT. This allows the macro to be used in if-then-elses without curly braces. Build for ft32. Committed as obvious. Thanks, - Tom [ft32] Remove semicolon after ASM_OUTPUT_ADDR_VEC_ELT 2017-11-23 Tom de Vries

[libobjc, committed] Wrap CLASS_TABLE_HASH in do {} while (0)

2017-11-23 Thread Tom de Vries
Hi, this patch wraps CLASS_TABLE_HASH in "do {} while (0)". This allows the macro to be used in if-then-elses without curly braces. Build libobcj for x86_64. Committed as obvious. Thanks, - Tom [libobjc] Wrap CLASS_TABLE_HASH in do {} while (0) 2017-11-23 Tom de Vries

[sh, committed] Wrap ASM_OUTPUT_ADDR_VEC_ELT in do {} while (0)

2017-11-23 Thread Tom de Vries
Hi, this patch wraps the sh version of ASM_OUTPUT_ADDR_VEC_ELT in "do {} while (0)". This allows the macro to be used in if-then-elses without curly braces. Build for sh. Committed as obvious. Thanks, - Tom [sh] Wrap ASM_OUTPUT_ADDR_VEC_ELT in do {} while (0) 2017-11-23 To

[libgccjit, committed] Wrap RETURN_NULL_IF_FAIL_NONNULL_NUMERIC_TYPE in JIT_{BEGIN,END}_STMT.

2017-11-23 Thread Tom de Vries
RETURN_NULL_IF_FAIL_NONNULL_NUMERIC_TYPE in JIT_{BEGIN,END}_STMT. 2017-11-23 Tom de Vries * libgccjit.c (RETURN_NULL_IF_FAIL_NONNULL_NUMERIC_TYPE): Wrap in JIT_{BEGIN,END}_STMT. --- gcc/jit/libgccjit.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/gcc/jit/libgccjit.c b/gcc/jit/libgccjit.c index

[RFC] Generate sub-tests from libgomp.c/for-{3,5,6}.c

2017-11-24 Thread Tom de Vries
[ add missing cc gcc-patches ] On 11/24/2017 04:31 PM, Tom de Vries wrote: Hi, The libgomp offloading tests libgomp.c/for-{3,5,6}.c are monolithic tests which test lots of subtests, and if any of them fail, it would be good to be able to quickly known which subtest(s) fail. [ And if you fix

[testsuite, committed] Require effective target alloca for pr82875.c

2017-12-03 Thread Tom de Vries
-end/82875 */ +/* { dg-do compile } */ +/* { dg-options "-ftree-ter" } */ + +const int a = 100; + +void +foo (void) +{ + int c[a]; +} Hi, this patch requires effective target alloca for pr82875.c. Committed. Thanks, - Tom Require effective target alloca for pr82875.c 2017-12-04 To

[RFC] Add means to split dump file into several files -- Use in lra

2017-12-07 Thread Tom de Vries
he lra_end dump files ends with ";; Function ...", which should be in the next lra_start dump file. Once we enable this using a switch we can probably do better. Any comments? Thanks, - Tom Add dump_bump 2017-12-07 Tom de Vries * dumpfile.c (DUMP_FILE_INFO): Add inits for bump

Re: [PATCH, PR81844] Fix condition folding in c_parser_omp_for_loop

2017-12-10 Thread Tom de Vries
On 09/14/2017 12:55 PM, Jakub Jelinek wrote: Ok with those changes Hi, this PR has been fixed on trunk ( https://gcc.gnu.org/viewcvs/gcc?view=revision&revision=252873 ). We cannot count this PR as a 6/7 regression, because this test-case has been failing since fopenmp was introduced in 4.2

Re: patch to fix PR82353

2017-12-13 Thread Tom de Vries
On 10/16/2017 10:38 PM, Vladimir Makarov wrote: This is another version of the patch to fix    https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82353 The patch was successfully bootstrapped on x86-64 with Go and Ada. Committed as rev. 253796. Hi Vladimir, AFAIU this bit of the patch makes sure

[testsuite] Require effective target weak_undefined for visibility-22.c

2017-12-14 Thread Tom de Vries
of timing out due to random code execution). OK for trunk? Thanks, - Tom Require effective target weak_undefined for visibility-22.c 2017-12-14 Tom de Vries * lib/target-supports.exp (check_effective_target_weak_undefined): New proc. * gcc.dg/visibility-22.c: Require effect

Re: [testsuite] Require effective target weak_undefined for visibility-22.c

2017-12-14 Thread Tom de Vries
On 12/14/2017 02:47 PM, Jakub Jelinek wrote: On Thu, Dec 14, 2017 at 02:40:12PM +0100, Tom de Vries wrote: --- a/gcc/testsuite/gcc.dg/visibility-22.c +++ b/gcc/testsuite/gcc.dg/visibility-22.c @@ -1,6 +1,7 @@ /* PR target/32219 */ /* { dg-do run } */ /* { dg-require-visibility

Re: [testsuite] Require effective target weak_undefined for visibility-22.c

2017-12-14 Thread Tom de Vries
On 12/14/2017 03:18 PM, Rainer Orth wrote: Hi Jakub, On Thu, Dec 14, 2017 at 03:09:02PM +0100, Tom de Vries wrote: On 12/14/2017 02:47 PM, Jakub Jelinek wrote: On Thu, Dec 14, 2017 at 02:40:12PM +0100, Tom de Vries wrote: --- a/gcc/testsuite/gcc.dg/visibility-22.c +++ b/gcc/testsuite/gcc.dg

[PATCH, PR83327] Fix liveness analysis in lra for spilled-into hard regs

2017-12-15 Thread Tom de Vries
[ was: Re: patch to fix PR82353 ] On 12/14/2017 06:01 PM, Vladimir Makarov wrote: On 12/13/2017 07:34 AM, Tom de Vries wrote: On 10/16/2017 10:38 PM, Vladimir Makarov wrote: This is another version of the patch to fix     https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82353 The patch was

Re: [PATCH][GRAPHITE] Fix PR71351

2017-12-19 Thread Tom de Vries
On 09/21/2017 12:07 PM, Richard Biener wrote: -exit_edge = create_empty_if_region_on_edge (entry_edge, - unshare_expr (cond_expr)); This removes the fix for PR70045: ... diff --git a/gcc/graphite-isl-ast-to-gimple.c b/gcc/graphite-isl-ast-to-gi

Re: [PATCH][GRAPHITE] Fix PR71351

2017-12-19 Thread Tom de Vries
On 12/19/2017 02:05 PM, Richard Biener wrote: On Tue, 19 Dec 2017, Tom de Vries wrote: On 09/21/2017 12:07 PM, Richard Biener wrote: -exit_edge = create_empty_if_region_on_edge (entry_edge, - unshare_expr (cond_expr)); This removes the fix

[PATCH, PR83423] Don't call targetm.calls.static_chain in non-static function

2017-12-19 Thread Tom de Vries
)) return NULL; ... and the patch moves this test to a new function rtx_for_static_chain and uses that function instead of targetm.calls.static_chain in the backend, to fix this problem. Bootstrapped and reg-tested on x86_64. OK for trunk? Thanks, - Tom Don't call targetm.calls.st

Re: [PATCH,PTX] Add support for CUDA 9

2017-12-19 Thread Tom de Vries
On 12/20/2017 12:25 AM, Cesar Philippidis wrote: In CUDA 9, Nvidia removed support for treating the labels of functions as generic address spaces as part of their PTX 6.0 changes. More specifically, :

Re: [RFC] Add means to split dump file into several files -- Use in lra

2017-12-20 Thread Tom de Vries
ump 2017-12-20 Tom de Vries * dumpfile.c (DUMP_FILE_INFO): Add inits for bump and bump_name fields. (get_dump_file_name): Handle bump and bump_name fields. (dump_start): Handle bump field. (gcc::dump_manager::dump_bump, dump_bump) (gcc::dump_manager::reset_dump_bump, reset_dump

[openacc, committed] Simplify fold_internal_goacc_dim

2017-12-20 Thread Tom de Vries
Hi, this patch simplifies fold_internal_goacc_dim. Bootstrapped and reg-tested on x86_64. Committed as trivial. Thanks, - Tom Simplify fold_internal_goacc_dim 2017-12-20 Tom de Vries * gimple-fold.c (fold_internal_goacc_dim): Simplify. --- gcc/gimple-fold.c | 20

Re: [PATCH,PTX] Add support for CUDA 9

2017-12-20 Thread Tom de Vries
On 12/20/2017 11:59 PM, Cesar Philippidis wrote: On 12/19/2017 04:39 PM, Tom de Vries wrote: On 12/20/2017 12:25 AM, Cesar Philippidis wrote: In CUDA 9, Nvidia removed support for treating the labels of functions as generic address spaces as part of their PTX 6.0 changes. More specifically

Re: [PATCH] Add LVAL argument to c_fully_fold* and propagate it through (PR c/66618, PR c/69960)

2017-12-25 Thread Tom de Vries
On 11/09/2017 11:54 AM, Jakub Jelinek wrote: On Wed, Nov 08, 2017 at 06:57:55PM +0100, Jakub Jelinek wrote: On Wed, Nov 08, 2017 at 06:51:34PM +0100, Marek Polacek wrote: Ok, so like this if it passes bootstrap/regtest? Changes from the last patch: 1) false instead of lval for COMPOUND_EXPR an

[PATCH, PR82391] Fold acc_on_device with const arg

2017-12-26 Thread Tom de Vries
ice onto the builtin function. Bootstrapped and reg-tested on x86_64. Build and reg-tested for x86_64 with nvptx accelerator. OK for trunk? Thanks, - Tom Fold acc_on_device with const arg 2017-12-22 Tom de Vries PR libgomp/82391 * omp-offload.c (fold_builtin_acc_on_device_cst_arg

[testsuite, committed] Use relative line number in unroll-5.c

2017-12-26 Thread Tom de Vries
al constant less than or equal to" } */ + for (unsigned long i = 1; i <= 8; ++i) +bar(i); +} Hi, this patch changes the absolute line number into a relative one. Tested on x86_64 and committed. Thanks, - Tom Use relative line number in unroll-5.c 2017-12-26 Tom de Vries *

[nvptx, committed] Disable -gstatement-frontiers for nvptx

2017-12-27 Thread Tom de Vries
, - Tom Disable -gstatement-frontiers for nvptx 2017-12-27 Tom de Vries * config/nvptx/nvptx.c (nvptx_option_override): Disable -gstatement-frontiers. --- gcc/config/nvptx/nvptx.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/gcc/config/nvptx/nvptx.c b/gcc/config/nvptx/nvptx.c index

[libgomp, testsuite, committed] Workaround PR83046 in gang-static-2.c

2017-12-27 Thread Tom de Vries
Hi, this patches fixes a problem in the gang-static-2.c test-case, which works around the PR83046 ICE, re-enabling the test-case. Tested libgomp for x86_64 with nvptx accelerator. Committed. Thanks, - Tom Workaround PR83046 in gang-static-2.c 2017-12-27 Tom de Vries PR c++/83046

Re: [PATCH,PTX] Add support for CUDA 9

2017-12-27 Thread Tom de Vries
On 12/21/2017 06:19 PM, Cesar Philippidis wrote: My test results are somewhat inconsistent. On MG's build servers, there are no regressions in CUDA 8. Ack. On my laptop, there are fewer regressions in CUDA 9, than CUDA 8. If the patch causes regressions for either cuda 8 or cuda 9, then th

[libgomp, openacc, openmp, PR83046] Prune removed funcs from offload table

2017-12-28 Thread Tom de Vries
tstrapped and reg-tested on x86_64. Build and reg-tested on x86_64 with nvptx accelerator. OK for trunk? Thanks, - Tom Prune removed funcs from offload table 2017-12-27 Tom de Vries PR libgomp/83046 * lto-cgraph.c (output_offload_tables): Remove offload_funcs entries that no longer have

Re: [nvptx, committed] Disable -gstatement-frontiers for nvptx

2017-12-29 Thread Tom de Vries
On 12/29/2017 05:12 AM, Alexandre Oliva wrote: On Dec 27, 2017, Tom de Vries wrote: .loc file_index line_number column_position so this causes ptxas errors when compiling something for nvptx with -g, which breaks the nvptx build. What do the errors look like? The nvptx-none build

Re: [libgomp, openacc, openmp, PR83046] Prune removed funcs from offload table

2017-12-29 Thread Tom de Vries
SERVE_P to 1 on the fndecl, so that we then stream that flag. Done. Bootstrapped and reg-tested on x86_64. Build and reg-tested for x86_64 with nvptx accelerator. OK for trunk? Thanks, - Tom Prune removed funcs from offload table 2017-12-27 Tom de Vries PR libgomp/83046 * omp-expand.c (e

[RFC] Add vec::ordered_remove_if

2017-12-29 Thread Tom de Vries
[ was: Re: [libgomp, openacc, openmp, PR83046] Prune removed funcs from offload table ] On 12/28/2017 05:06 PM, Jakub Jelinek wrote: On Thu, Dec 28, 2017 at 04:53:29PM +0100, Tom de Vries wrote: --- a/gcc/lto-cgraph.c +++ b/gcc/lto-cgraph.c @@ -,6 +,16 @@ output_offload_tables (void

[testsuite, PR83612] Fix 'memory cannot be printed' in gcc.dg/ubsan/object-size-9.c

2017-12-30 Thread Tom de Vries
ttribute to the variable to ensure that the memory after the variable is accessible. Tested on x86_64. Committed as obvious. Thanks, - Tom Fix 'memory cannot be printed' in gcc.dg/ubsan/object-size-9.c 2017-12-30 Tom de Vries PR testsuite/83612 * gcc.dg/ubsan/object-size-9 (t)

Re: [libgomp, openacc, openmp, PR83046] Prune removed funcs from offload table

2017-12-30 Thread Tom de Vries
On 12/30/2017 10:54 AM, Jakub Jelinek wrote: On Fri, Dec 29, 2017 at 02:07:49PM +0100, Tom de Vries wrote: --- a/gcc/lto-streamer-out.c +++ b/gcc/lto-streamer-out.c @@ -41,6 +41,7 @@ along with GCC; see the file COPYING3. If not see #include "builtins.h" #include "g

[PATCH] Add VEC_ORDERED_REMOVE_IF

2018-01-05 Thread Tom de Vries
[ was: Re: [RFC] Add vec::ordered_remove_if ] On 01/02/2018 03:08 PM, Richard Biener wrote: On Fri, Dec 29, 2017 at 2:55 PM, Tom de Vries wrote: [ was: Re: [libgomp, openacc, openmp, PR83046] Prune removed funcs from offload table ] On 12/28/2017 05:06 PM, Jakub Jelinek wrote: On Thu, Dec

[PATCH, PR82428] Add __builtin_goacc_{gang,worker,vector}_{id,size}

2018-01-06 Thread Tom de Vries
apped and reg-tested on x86_64. Build and reg-tested on x86_64 with nvptx accelerator. OK for trunk? Thanks, - Tom Add __builtin_goacc_{gang,worker,vector}_{id,size} 2017-12-30 Tom de Vries PR libgomp/82428 * builtins.def (DEF_GOACC_BUILTIN_ONLY): Define. * omp-builtins.def (BUILT_IN_GOACC

Re: [PATCH, PR82428] Add __builtin_goacc_{gang,worker,vector}_{id,size}

2018-01-07 Thread Tom de Vries
On 01/06/2018 12:36 PM, Jakub Jelinek wrote: On Sat, Jan 06, 2018 at 09:21:59AM +0100, Tom de Vries wrote: this patch adds the following builtins in C/C++: - __builtin_goacc_gang_id - __builtin_goacc_worker_id - __builtin_goacc_vector_id - __builtin_goacc_gang_size - __builtin_goacc_worker_size

Re: [PING 2][PATCH] enhance -Wrestrict to handle string built-ins (PR 78918)

2018-01-08 Thread Tom de Vries
On 12/17/2017 01:01 AM, Martin Sebor wrote: * c-c++-common/Wrestrict.c: New test. 681/* The following doesn't overlap but it should trigger -Wstrinop-ovewrflow 682 for writing past the end. */ 683T ("012", a + sizeof a, a); For nvptx, the warning actually shows

[testsuite] Require alloca for some test-cases

2018-01-08 Thread Tom de Vries
Hi, This patch requires alloca for some test-cases. Tested on x86_64 and committed. Thanks, - Tom Require alloca for some test-cases 2018-01-08 Tom de Vries * c-c++-common/builtins.c: Require effective target alloca. * gcc.dg/Wrestrict.c: Same. * gcc.dg/tree-ssa/loop-interchange-15.c

<    3   4   5   6   7   8   9   10   11   12   >