Re: [PATCH 2/n] [PR tree-optimization/78496] Simplify ASSERT_EXPRs to facilitate jump threading

2017-05-06 Thread Andrew Pinski
On Sat, May 6, 2017 at 4:55 PM, Andrew Pinski wrote: > On Sat, May 6, 2017 at 8:03 AM, Jeff Law wrote: >> This is the 2nd of 3-5 patches to address pr78496. >> >> Jump threading will examine ASSERT_EXPRs as it walks the IL and record >> information from those ASSERT_EXPRs into the available expre

Re: [PATCH 2/n] [PR tree-optimization/78496] Simplify ASSERT_EXPRs to facilitate jump threading

2017-05-06 Thread Andrew Pinski
On Sat, May 6, 2017 at 8:03 AM, Jeff Law wrote: > This is the 2nd of 3-5 patches to address pr78496. > > Jump threading will examine ASSERT_EXPRs as it walks the IL and record > information from those ASSERT_EXPRs into the available expression and > const/copies tables where they're later used to

[PATCH] PR80280: Fix quoting of candidate functions

2017-05-06 Thread Volker Reichelt
Hi, the following patch fixes some wrong quoting that was introduced by Martin's patch for PR translation/80280, see https://gcc.gnu.org/viewcvs/gcc?view=revision&revision=247607 Consider the following testcase: struct A {}; bool b = !A(); On trunk we currently get the following diagnostic: b

Re: [PATCH v4 0/12] [i386] Improve 64-bit Microsoft to System V ABI pro/epilogues

2017-05-06 Thread Daniel Santos
On 05/05/2017 03:56 AM, Daniel Santos wrote: On 05/02/2017 05:40 AM, Kai Tietz wrote: Right, and Wine people will tell, if something doesn't work for them. So ok for me too. Kai Well, I haven't re-run these tests in a few months, but I got 272 failed wine tests with gcc 7.1 and 234 with my pat

Re: [PATCH 2/n] [PR tree-optimization/78496] Simplify ASSERT_EXPRs to facilitate jump threading

2017-05-06 Thread Jeff Law
On 05/06/2017 09:38 AM, Trevor Saunders wrote: +simplify_assert_expr_using_ranges (gimple *stmt) +{ + return false; I assume you didn't mean to leave that in? No I didn't. I used to to extract the testcases. Ugh. jeff

Re: [PATCH, ARM] Add a new target hook to compute the frame layout

2017-05-06 Thread Bernd Edlinger
On 05/05/17 16:59, Richard Earnshaw (lists) wrote: > On 05/09/16 17:43, Bernd Edlinger wrote: >> Hi Richard, >> >> what do you think of this patch, is it OK (with the suggested wording)? >> > > Bernd, > > Apologies, this seems to have fallen through a crack. > > I'm happy with this. Does it still

Re: [PATCH 2/n] [PR tree-optimization/78496] Simplify ASSERT_EXPRs to facilitate jump threading

2017-05-06 Thread Trevor Saunders
> +simplify_assert_expr_using_ranges (gimple *stmt) > +{ > + return false; I assume you didn't mean to leave that in? Trev

[PING][patch] have "would be stringified in traditional C" warning in libcpp/macro.c be controlled by -Wtraditional

2017-05-06 Thread Eric Gallager
Pinging this: https://gcc.gnu.org/ml/gcc-patches/2017-03/msg01325.html On 3/24/17, Eric Gallager wrote: > It seemed odd to me that gcc was issuing a warning about compatibility > with traditional C that I couldn't turn off by pushing/popping > -Wtraditional over the problem area, so I made the at

[PATCH 2/n] [PR tree-optimization/78496] Simplify ASSERT_EXPRs to facilitate jump threading

2017-05-06 Thread Jeff Law
This is the 2nd of 3-5 patches to address pr78496. Jump threading will examine ASSERT_EXPRs as it walks the IL and record information from those ASSERT_EXPRs into the available expression and const/copies tables where they're later used to simplify conditionals. We're missing a couple things

Re: [PATCH 1/N] Introduce dump_flags_t type and use it instead of int type.

2017-05-06 Thread Trevor Saunders
On Fri, May 05, 2017 at 01:38:21PM +0200, Richard Biener wrote: > On Fri, May 5, 2017 at 12:41 PM, Martin Liška wrote: > > Hello. > > > > There's first patch that just defines dump_flags_t as uint64_t and changes > > all > > corresponding interfaces that do use it. There's a problematic impact th

New Spanish PO file for 'gcc' (version 7.1.0)

2017-05-06 Thread Translation Project Robot
Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'gcc' has been submitted by the Spanish team of translators. The file is available at: http://translationproject.org/latest/gcc/es.po (This file, 'gcc-7.1.0.es.po', has just

Re: [PATCH] c++ parser: fix-it hints for wrong usage of 'friend' and 'auto'

2017-05-06 Thread Volker Reichelt
On 2 May, Martin Sebor wrote: > On 04/29/2017 04:23 PM, Volker Reichelt wrote: >> Hi, >> >> the following patch adds fix-it hints to the C++ parser for two wrongly >> used keywords detected in cp_parser_decl_specifier_seq. >> >> Bootstrapped and regtested on x86_64-pc-linux-gnu. >> >> OK for trunk

Re: [PATCH, GCC/ARM, Stage 1] PR71607: Fix ICE when loading constant

2017-05-06 Thread Tom de Vries
On 04/20/2017 11:54 AM, Prakhar Bahuguna wrote: diff --git a/gcc/testsuite/gcc.target/arm/tls-disable-literal-pool.c b/gcc/testsuite/gcc.target/arm/tls-disable-literal-pool.c new file mode 100644 index 000..fe14a6b132c --- /dev/null +++ b/gcc/testsuite/gcc.target/arm/tls-disable-literal-

[testsuite, committed] Add absolute line number comment in gcc.target/avr/progmem-error-1.cpp

2017-05-06 Thread Tom de Vries
Hi, this patchs adds an absolute line number comment in gcc.target/avr/progmem-error-1.cpp. Committed as obvious. Thanks, - Tom Add absolute line number comment in gcc.target/avr/progmem-error-1.cpp 2017-05-05 Tom de Vries * gcc.target/avr/progmem-error-1.cpp: Add abolute line number co

[testsuite, committed] Replace absolute line numbers in gcc.target/powerpc

2017-05-06 Thread Tom de Vries
Hi, this patch replaces absolute line numbers in gcc.target/powerpc. Tested on ppc. Committed as obvious. Thanks, - Tom Replace absolute line numbers in gcc.target/powerpc 2017-05-05 Tom de Vries PR testsuite/80557 * gcc.target/powerpc/altivec-macros.c: Replace absolute line numbers. *

[testsuite, committed] Replace absolute line numbers in gcc.target/spu

2017-05-06 Thread Tom de Vries
Hi, this patch replaces absolute line numbers in gcc.target/spu. Tested on spu. Committed as obvious. Thanks, - Tom Replace absolute line numbers in gcc.target/spu 2017-05-05 Tom de Vries PR testsuite/80557 * gcc.target/spu/Wmain.c: Replace absolute line numbers. * gcc.target/spu/intri

Re: [patch,avr,committed]: Remove flag_strict_overflow from avr.md

2017-05-06 Thread Richard Sandiford
Richard Biener writes: > On Fri, 5 May 2017, Richard Sandiford wrote: >> Richard Biener writes: >> > On Fri, 5 May 2017, Georg-Johann Lay wrote: >> >> On 05.05.2017 13:04, Richard Biener wrote: >> >> > On Fri, 5 May 2017, Georg-Johann Lay wrote: >> >> > >> >> > > Applied this addendum to r247495

[testsuite, committed] Replace absolute line numbers in gcc.target/arm

2017-05-06 Thread Tom de Vries
Hi, this patch replaces absolute line numbers in gcc.target/arm. Tested on arm. Committed as obvious. Thanks, - Tom Replace absolute line numbers in gcc.target/arm 2017-05-05 Tom de Vries PR testsuite/80557 * gcc.target/arm/pr69180.c: Replace absolute line numbers. --- gcc/testsuite/g

[testsuite, committed] Replace absolute line numbers in gcc.target/aarch64

2017-05-06 Thread Tom de Vries
Hi, this patch replaces absolute line numbers in gcc.target/aarch64. Tested on aarch64. Committed as obvious. Thanks, - Tom Replace absolute line numbers in gcc.target/aarch64 2017-05-05 Tom de Vries PR testsuite/80557 * gcc.target/aarch64/spellcheck_1.c: Replace absolute line numbers.

Re: [patch, testsuite, PR80606] Remove default_packed lines from i386/avx-vtestp{d,s}*

2017-05-06 Thread Uros Bizjak
On Sat, May 6, 2017 at 9:12 AM, Tom de Vries wrote: > Hi, > > The test-cases i386/avx-vtestp{d,s}* contain lines guarded with target > default_packed, using absolute line numbers 150 and 164. The absolute line > numbers intend to refer to the position of the packed attribute in unions > ieee754_f

[committed, testsuite] Replace absolute line numbers in g++.dg/{debug,goacc}

2017-05-06 Thread Tom de Vries
Hi, this patch replaces absolute line numbers in g++.dg/{debug,goacc}. Tested on x86_64. Committed as obvious. Thanks, - Tom Replace absolute line numbers in g++.dg/{debug,goacc} 2017-05-05 Tom de Vries PR testsuite/80557 * g++.dg/debug/dwarf2/dwarf2-1.C: Replace absolute line numbers.

Re: [PATCH 1/3] Remove support for obsolete x86 -malign-foo options

2017-05-06 Thread Uros Bizjak
On Tue, Apr 18, 2017 at 8:30 PM, Denys Vlasenko wrote: > 2017-04-18 Denys Vlasenko > > * config/i386/i386-common.c (ix86_handle_option): Remove support > for obsolete -malign-loops, -malign-jumps and -malign-functions > options. > * config/i386/i386.opt: Likewise. > Index: gcc/c

[patch, testsuite, PR80606] Remove default_packed lines from i386/avx-vtestp{d,s}*

2017-05-06 Thread Tom de Vries
Hi, The test-cases i386/avx-vtestp{d,s}* contain lines guarded with target default_packed, using absolute line numbers 150 and 164. The absolute line numbers intend to refer to the position of the packed attribute in unions ieee754_float and ieee754_double in m128-check.h, but they do not ma

[PATCH, i386]: Some trivial cleanups

2017-05-06 Thread Uros Bizjak
2017-05-06 Uros Bizjak * config/i386/i386.c (ext_80387_constant_init): Do not explicitly initialize to zero. (init_regs): Remove declaration. (function_arg_advance_32): Initialize error_p as boolean variable. Bootstrapped and regression tested on x86_64-linux-gnu. Committed to