Re: [PATCH] Fix PR64822: incorrect folding of bitfield in union on big endian targets

2015-02-03 Thread Jakub Jelinek
On Wed, Feb 04, 2015 at 10:45:11AM +0800, Thomas Preud'homme wrote: > 2015-01-30 Thomas Preud'homme > > PR middle-end/62103 > * tree-ssa-sccvn.c (fully_constant_vn_reference_p): Use TYPE_PRECISION > to compute size of referenced value in the constant case. > > 2015-01-30 Tho

[PR64817-related 3/3] simplify xor of (and or ior) of xor

2015-02-03 Thread Alexandre Oliva
PR64817's testcase creates a long chain of XOR of AND of XOR of AND of that our rtl simplifiers can't simplify, although they are simplifyable. combine manages to simplify the generated insns that perform the computation represented by such chains, but simplify-rtx doesn't. That's because combine

[PR64817-related 2/3] don't alloc rtl when failing to simplify XOR of AND

2015-02-03 Thread Alexandre Oliva
We have a problem in simplify_binary_operation_1 that causes memory waste in general, and memory explosion in pathological testcases such as that of PR64817, with large exprs amounting to XOR of AND of XOR of AND of... I believe rtl simplifiers are not supposed to allocate rtl before committing to

[PR64817-related 1/3] fix debug expr expand of compares

2015-02-03 Thread Alexandre Oliva
PR64817 is lucky that the compare in the testcase was <0 rather than >0. expand_debug_expr used to take the signedness of the expr from the result type, rather than from the operand types, so it the a < 0 test was expanded as LTU rather than LT, and LTU compares with zero are always false, so we s

Re: [PATCH] Add top-level config support for gold mips target

2015-02-03 Thread Cary Coutant
Ping^3. Should I be addressing this to someone else? -cary On Mon, Dec 1, 2014 at 2:15 PM, Cary Coutant wrote: > Ping^2. > > -cary > > On Wed, Oct 29, 2014 at 12:04 PM, Cary Coutant wrote: >> Ping? >> >> On Mon, Oct 20, 2014 at 10:31 AM, Cary Coutant wrote: >>> This patch adds support for the

Re: PR lto/64837: lto plugin doesn't call ld_plugin_release_input_file

2015-02-03 Thread Cary Coutant
The plugin is not supposed to call release_input_file from the claim_file handler. That interface is only for releasing a file descriptor obtained via get_input_file during the all_symbols_read callback. When the linker calls the claim_file handler, the file descriptor is open, and the plugin is re

Re: r219977 - in /trunk/gcc: ChangeLog config/rs600...

2015-02-03 Thread David Edelsohn
On Tue, Feb 3, 2015 at 5:55 PM, Andreas Schwab wrote: > FAIL: gcc.dg/builtins-58.c scan-assembler-not pow > > $ grep pow builtins-58.s > .machine power4 Any suggestions? - David

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

2015-02-03 Thread Bin.Cheng
On Wed, Feb 4, 2015 at 12:28 AM, Jeff Law wrote: > On 02/03/15 01:29, Bin.Cheng wrote: >> >> >> Hmm, if I understand correctly, it's a code size regression, so I >> don't think it's appropriate to adapt the test case. Either the patch >> or something else in GCC is doing wrong, right? >> >> Hi Al

RE: [PATCH] Fix PR64822: incorrect folding of bitfield in union on big endian targets

2015-02-03 Thread Thomas Preud'homme
> From: Jakub Jelinek [mailto:ja...@redhat.com] > Sent: Thursday, January 29, 2015 6:39 PM > > You should mention > PR middle-end/62103 Right, please find the new ChangeLog entries below: 2015-01-30 Thomas Preud'homme PR middle-end/62103 * tree-ssa-sccvn.c (fully_const

[c++-concepts] Bring constraints in line with spec

2015-02-03 Thread Braden Obrzut
This is a large patch mostly written by Andrew to change how constraints are stored. It brings the implementation more in line with the specification and simplifies some parts. I'm not entirely sure about the change to cp_parser_template_argument_list. What happens here is when a template t

RE: [PATCH, FT32] initial support

2015-02-03 Thread James Bowman
> optabs.c's expand_abs_nojump already knows this trick: > > /* If this machine has expensive jumps, we can do integer absolute >value of X as (((signed) x >> (W-1)) ^ x) - ((signed) x >> (W-1)), >where W is the width of MODE. */ > > So if you define BRANCH_COST to be 2 or more ther

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

2015-02-03 Thread H.J. Lu
On Tue, Feb 3, 2015 at 2:19 PM, Jakub Jelinek wrote: > On Tue, Feb 03, 2015 at 02:03:14PM -0800, H.J. Lu wrote: >> So we aren't SYMBOL_REF_EXTERNAL_P nor >> SYMBOL_REF_LOCAL_P. What do we reference? > > That is reasonable. There is no guarantee the extern weak symbol is local, > it could very we

Re: Fix PR64876, regressions in powerpc64 Go testsuite

2015-02-03 Thread Alan Modra
On Tue, Feb 03, 2015 at 11:14:49AM -0500, David Edelsohn wrote: > On Tue, Feb 3, 2015 at 8:57 AM, Alan Modra wrote: > > PR target/64876 > > * config/rs6000/rs6000.c (chain_already_loaded): New function. > > (rs6000_call_aix): Use it. > > Okay with Jakub's suggested change.

[Ping] Re: [PATCH 1/3] Replace MD_REDIRECT_BRANCH with TARGET_CAN_FOLLOW_JUMP

2015-02-03 Thread Kaz Kojima
Kaz Kojima wrote: > 2015-01-27 Joern Rennecke > Kaz Kojima > > PR target/64761 > * config/sh/sh-protos.h (sh_can_redirect_branch): Don't declare. > * config/sh/sh.c (TARGET_CAN_FOLLOW_JUMP): Redefine. > (sh_can_redirect_branch): Rename to ... > (sh_can

[RFC testsuite] Fix PR64850, tweak acc_on_device* tests

2015-02-03 Thread Kaz Kojima
Hi, Several goacc/acc_on_device tests fail for a few targets: hppa2.0w-hp-hpux11.11 (PR testsuite/64850) https://gcc.gnu.org/ml/gcc-testresults/2015-01/msg02659.html m68k-unknown-linux-gnu https://gcc.gnu.org/ml/gcc-testresults/2015-01/msg02960.html sh4-unknown-linux-gnu https://gcc.gnu.org/ml/

Re: Fix PR64876, regressions in powerpc64 Go testsuite

2015-02-03 Thread Alan Modra
On Tue, Feb 03, 2015 at 03:08:01PM +0100, Jakub Jelinek wrote: > On Wed, Feb 04, 2015 at 12:27:35AM +1030, Alan Modra wrote: > > @@ -33002,7 +33092,9 @@ rs6000_call_aix (rtx value, rtx func_desc, rtx fla > > originally direct, the 3rd word has not been written since no > > trampol

Re: r219977 - in /trunk/gcc: ChangeLog config/rs600...

2015-02-03 Thread Andreas Schwab
FAIL: gcc.dg/builtins-58.c scan-assembler-not pow $ grep pow builtins-58.s .machine power4 Andreas. -- Andreas Schwab, sch...@linux-m68k.org GPG Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276 4ED5 "And now for something completely different."

Re: [PATCH, FT32] initial support

2015-02-03 Thread Joseph Myers
This patch is missing pieces such as Texinfo documentation (in invoke.texi for target-specific options, at least) and config-list.mk update so automatic builders verify that this target builds OK. See "Back End" in sourcebuild.texi and make sure that you have everything relevant. It's a good idea

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

2015-02-03 Thread Jakub Jelinek
On Tue, Feb 03, 2015 at 02:03:14PM -0800, H.J. Lu wrote: > So we aren't SYMBOL_REF_EXTERNAL_P nor > SYMBOL_REF_LOCAL_P. What do we reference? That is reasonable. There is no guarantee the extern weak symbol is local, it could very well be non-local. All that you know about the symbols is that i

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

2015-02-03 Thread H.J. Lu
On Tue, Feb 3, 2015 at 1:35 PM, Sriraman Tallam wrote: > On Tue, Feb 3, 2015 at 1:29 PM, H.J. Lu wrote: >> On Tue, Feb 3, 2015 at 1:20 PM, Sriraman Tallam wrote: >>> On Tue, Feb 3, 2015 at 11:36 AM, Jakub Jelinek wrote: On Tue, Feb 03, 2015 at 11:25:38AM -0800, Sriraman Tallam wrote: >

[PATCH, committed] fix Fortran docs

2015-02-03 Thread Steve Kargl
I've committed the following typo fix to both 4.9 and 5.0. Index: ChangeLog === --- ChangeLog (revision 220347) +++ ChangeLog (working copy) @@ -1,3 +1,6 @@ +2015-02-03 Steven G. Kargl + + * intrinsic.texi (CO_ASSOCIATED)

Remove libgo/go/go/types/testdata

2015-02-03 Thread Ian Lance Taylor
The libgo/go/go/types directory was removed in July 2013. Unfortunately I accidentally left the testdata directory behind. This patch removes it. Committed to mainline. Ian diff -r 85d0b689bd17 libgo/go/go/types/testdata/builtins.src --- a/libgo/go/go/types/testdata/builtins.src Mon Feb 02 19:

Re: [Patch, fortran] PR 64757 - [5 Regression] ICE in fold_convert_loc, at fold-const.c:2353

2015-02-03 Thread Paul Richard Thomas
Dear Dominique, I have fixed all the problems except the last one. For that case, the other brand gives type_to_class_30.f90(19): error #7822: Variables containing ultimate allocatable array components are forbidden from appearing directly in input/output lists. print *, TestReference([Test(99), T

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

2015-02-03 Thread Sriraman Tallam
On Tue, Feb 3, 2015 at 1:29 PM, H.J. Lu wrote: > On Tue, Feb 3, 2015 at 1:20 PM, Sriraman Tallam wrote: >> On Tue, Feb 3, 2015 at 11:36 AM, Jakub Jelinek wrote: >>> On Tue, Feb 03, 2015 at 11:25:38AM -0800, Sriraman Tallam wrote: This was the original patch to i386.c to let global accesses

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

2015-02-03 Thread H.J. Lu
On Tue, Feb 3, 2015 at 1:20 PM, Sriraman Tallam wrote: > On Tue, Feb 3, 2015 at 11:36 AM, Jakub Jelinek wrote: >> On Tue, Feb 03, 2015 at 11:25:38AM -0800, Sriraman Tallam wrote: >>> This was the original patch to i386.c to let global accesses take >>> advantage of copy relocations and avoid the

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

2015-02-03 Thread Sriraman Tallam
On Tue, Feb 3, 2015 at 11:36 AM, Jakub Jelinek wrote: > On Tue, Feb 03, 2015 at 11:25:38AM -0800, Sriraman Tallam wrote: >> This was the original patch to i386.c to let global accesses take >> advantage of copy relocations and avoid the GOT. >> >> >> @@ -13113,7 +13113,11 @@ legitimate_pic_address

[SH][committed] Use shorter atomic sequences if result values are unused

2015-02-03 Thread Oleg Endo
Hi, When the result values of atomic ops, such as the previous value of an atomic_fetch_add, are unused, it's possible to use shorter asm sequences on SH. The attached patch does that by checking the reg unused notes of the insns in split1 and replacing them with the simpler variants, if the resu

Re: [gofrontend-dev] GO tools for gccgo cross

2015-02-03 Thread Ian Lance Taylor
On Tue, Feb 3, 2015 at 11:52 AM, Lynn A. Boger wrote: > > I've been experimenting with the go tools and how to make them work for > cross gccgo builds. > > In golang I think there is just one 'go' tool and the cross build targets > are specified by the setting of GOARCH and GOOS. So why couldn't

Re: Merge current set of OpenACC changes from gomp-4_0-branch

2015-02-03 Thread Ilya Verbin
On 03 Feb 13:00, Julian Brown wrote: > On Tue, 3 Feb 2015 14:28:44 +0300 > Ilya Verbin wrote: > > On 27 Jan 14:07, Julian Brown wrote: > > > On Mon, 26 Jan 2015 17:34:26 +0300 > > > Ilya Verbin wrote: > > > > Here is my current patch, it works for OpenMP->MIC, but obviously > > > > will not work

GO tools for gccgo cross

2015-02-03 Thread Lynn A. Boger
Hi, I've been experimenting with the go tools and how to make them work for cross gccgo builds. In golang I think there is just one 'go' tool and the cross build targets are specified by the setting of GOARCH and GOOS. So why couldn't the same be done with gccgo? That means, on any given

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

2015-02-03 Thread Jakub Jelinek
On Tue, Feb 03, 2015 at 11:25:38AM -0800, Sriraman Tallam wrote: > This was the original patch to i386.c to let global accesses take > advantage of copy relocations and avoid the GOT. > > > @@ -13113,7 +13113,11 @@ legitimate_pic_address_disp_p (rtx disp) > return true; > } >else if (!

[PATCH, committed] jit: option-logging

2015-02-03 Thread David Malcolm
It seems prudent to have the jit's log files contain information on the values of the various options in use. This patch adds logging when options are changed, and logs the value of all options when a compile is started (and no-ops for the common case when no logging is enabled). Tested via "make

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

2015-02-03 Thread Sriraman Tallam
+davidxl +ccoutant On Tue, Feb 3, 2015 at 11:25 AM, Sriraman Tallam wrote: > On Thu, Dec 4, 2014 at 8:46 AM, H.J. Lu wrote: >> On Thu, Dec 4, 2014 at 4:44 AM, Uros Bizjak wrote: >>> On Wed, Dec 3, 2014 at 10:35 PM, H.J. Lu wrote: >>> > It would probably help reviewers if you pointed to

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

2015-02-03 Thread Sriraman Tallam
On Thu, Dec 4, 2014 at 8:46 AM, H.J. Lu wrote: > On Thu, Dec 4, 2014 at 4:44 AM, Uros Bizjak wrote: >> On Wed, Dec 3, 2014 at 10:35 PM, H.J. Lu wrote: >> It would probably help reviewers if you pointed to actual path submission [1], which unfortunately contains the explanation

Re: [patch+7.9] compile: Filter out -fpreprocessed

2015-02-03 Thread Mark Wielaard
On Tue, 2015-02-03 at 19:59 +0100, Jan Kratochvil wrote: > On Tue, 03 Feb 2015 19:50:40 +0100, Doug Evans wrote: > > On Fri, Jan 16, 2015 at 2:42 PM, Jan Kratochvil > > wrote: > > > [...] > > > It is wrong that gcc puts -fpreprocessed into DW_AT_producer - I may post > > > a gcc > > > patch for i

Re: [patch+7.9] compile: Filter out -fpreprocessed

2015-02-03 Thread Jan Kratochvil
On Tue, 03 Feb 2015 19:50:40 +0100, Doug Evans wrote: > On Fri, Jan 16, 2015 at 2:42 PM, Jan Kratochvil > wrote: > > [...] > > It is wrong that gcc puts -fpreprocessed into DW_AT_producer - I may post a > > gcc > > patch for it. > > Hi. > I wasn't aware there are now rules for what can and canno

Re: [PATCH, testsuite] Fix PR64796: bswap64 effective target should not cache its result

2015-02-03 Thread Jeff Law
On 01/27/15 02:36, Thomas Preud'homme wrote: As explained in PR64796, code for bswap64 effective target computes the answer once and then cache. However the result depends on the flags passed to the compiler and with --target_board it's possible to test several sets of flags. Besides, this cod

Re: [patch+7.9] compile: Filter out -fpreprocessed

2015-02-03 Thread Doug Evans
On Fri, Jan 16, 2015 at 2:42 PM, Jan Kratochvil wrote: > [...] > It is wrong that gcc puts -fpreprocessed into DW_AT_producer - I may post a > gcc > patch for it. Hi. I wasn't aware there are now rules for what can and cannot go in DW_AT_producer. DW_AT_producer has gone from being informational

Re: [PATCH, CHKP] Follow alias chain for decl visibility and aliases

2015-02-03 Thread Jeff Law
On 01/30/15 15:18, Ilya Enkovich wrote: Hi, This patch fixes two more cases where alias chain should be followed to emit correct assembler name for instrumented functions. Bootstrapped and tested on x86_64-unknown-linux-gnu. OK for trunk? Thanks, Ilya -- gcc/ 2015-01-30 Ilya Enkovich

Re: [RFC PATCH] Avoid most of the BUILT_IN_*_CHKP enum values

2015-02-03 Thread Jeff Law
On 01/27/15 07:27, Jakub Jelinek wrote: Hi! I've grepped for BUILT_IN_.*_CHKP in the sources and we actually need far fewer enum values than the 1204 that are being defined. This patch requires builtins.def to say explicitly (by using DEF_*BUILTIN_CHKP macro instead of corresponding DEF_*BUILTI

RE: [PATCH MIPS RFA] Regression cleanup for nan2008 toolchain

2015-02-03 Thread Moore, Catherine
> -Original Message- > From: Robert Suchanek [mailto:robert.sucha...@imgtec.com] > Sent: Monday, February 02, 2015 11:18 AM > To: Richard Sandiford > Cc: gcc-patches@gcc.gnu.org; Matthew Fortune; Moore, Catherine > Subject: RE: [PATCH MIPS RFA] Regression cleanup for nan2008 toolchain >

Re: Bug 62044 - [4.8/4.9 Regression] ICE in USE statement with RENAME for extended derived type

2015-02-03 Thread H.J. Lu
On Tue, Jan 27, 2015 at 1:09 PM, Paul Richard Thomas wrote: > Dear All, > > The highly embarrassing bug in mold = allocations to class entities > has been fixed in revisions 220140 and 220191 for trunk and 4.9 > respectively. The PR has been set as RESOLVED. > This bug fix may have caused: https

Re: [PATCH][ARM][PING] __ARM_FP & __ARM_NEON_FP defined when -march=armv7-m

2015-02-03 Thread Richard Earnshaw
On 06/01/15 09:40, Mantas Mikaitis wrote: > > Ping and changelog spaces removed. > > Thank you, > Mantas M. > > On 18/11/14 11:58, Richard Earnshaw wrote: >> On 18/11/14 11:30, Mantas Mikaitis wrote: >>> Incorrect predefinitions for certain target architectures. E.g. arm7-m >>> does not contain

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

2015-02-03 Thread Jeff Law
On 02/03/15 01:29, Bin.Cheng wrote: Hmm, if I understand correctly, it's a code size regression, so I don't think it's appropriate to adapt the test case. Either the patch or something else in GCC is doing wrong, right? Hi Alex, could you please file a PR with full dump information for trackin

Re: [C++ Patch/RFC] PR 64877

2015-02-03 Thread Jason Merrill
On 02/03/2015 11:14 AM, Paolo Carlini wrote: + /* Avoid -Waddress warnings (c++/64877). */ + TREE_NO_WARNING (pfn0) = 1; I'd check for ADDR_EXPR before doing this; OK with that change. Jason

Re: Fix PR64876, regressions in powerpc64 Go testsuite

2015-02-03 Thread David Edelsohn
On Tue, Feb 3, 2015 at 8:57 AM, Alan Modra wrote: > This fixes a large number of Go testsuite failures on powerpc64 ELFv1, > caused by loading r11 from a function descriptor and thus trashing the > value set up from CALL_EXPR_STATIC_CHAIN. So don't load r11 if it > already contains a useful value

Re: [C++ Patch/RFC] PR 64877

2015-02-03 Thread Paolo Carlini
Hi, On 02/03/2015 03:19 PM, Jason Merrill wrote: On 02/03/2015 05:45 AM, Paolo Carlini wrote: + if (TREE_CODE (pfn0) != ADDR_EXPR + || !decl_with_nonnull_addr_p (TREE_OPERAND (pfn0, 0))) I don't like duplicating the logic for when we might know the pfn is non-null; that see

Re: [PATCH] Fix combiner from accessing or writing out of bounds SET_N_REGS (PR other/63504)

2015-02-03 Thread H.J. Lu
On Mon, Feb 2, 2015 at 11:20 PM, Jakub Jelinek wrote: > On Mon, Feb 02, 2015 at 05:26:23PM -0600, Segher Boessenkool wrote: >> On Mon, Feb 02, 2015 at 07:54:46PM +0100, Jakub Jelinek wrote: >> > +/* Highest pseudo for which we track REG_N_SETS. */ >> > +static unsigned int reg_n_sets_max; >> >> O

[PATCH][ARM] PR target/64600 Fix another ICE with -mtune=xscale: properly sign-extend mask during constant splitting

2015-02-03 Thread Kyrill Tkachov
Hi all, The ICE in this PR occurs when -mtune=xscale triggers a particular path through arm_gen_constant during expand that creates a 0xf00f mask but for a 64-bit HOST_WIDE_INT doesn't sign extend it into 0xf00f that signifies the required -4081. It leaves it as 0xf00f (429

Re: [PATCH, FT32] initial support

2015-02-03 Thread Paolo Bonzini
On 03/02/2015 07:05, Andrew Pinski wrote: > Likewise of: > +(define_insn "abssi2" > + [(set (match_operand:SI 0 "register_operand" "=r") > + (abs:SI (match_operand:SI 1 "register_operand" "r"))) > + (clobber (match_scratch:SI 2 "=&r"))] > + "" > + "ashr.l\t%2,%1,31\;xor.l\t%0,%1,%2\;sub.l\t%

RFA: Tweak documentation of fma

2015-02-03 Thread Richard Sandiford
The original reason for this was to fix the missing space before "@var{z}". "Do a combined multiply ... and then" didn't sound quite right though: "and then" implies a separate step and so feels like a contradiction of "combined". Tested with "make doc html pdf". OK to install? Thanks, Richard

[ARM, committed] Fix rtl checking failure in thumb2_reorg

2015-02-03 Thread Richard Sandiford
The thumb2_reorg code: if (!OBJECT_P (src)) op0 = XEXP (src, 0); causes an rtl checking failure if SRC is an UNSPEC. This doesn't matter in practice without rtl checking since OP0 is only used if SRC is a unary or binary operator. This patch tightens the condition to reflect that.

Re: [C++ Patch/RFC] PR 64877

2015-02-03 Thread Jason Merrill
On 02/03/2015 05:45 AM, Paolo Carlini wrote: + if (TREE_CODE (pfn0) != ADDR_EXPR + || !decl_with_nonnull_addr_p (TREE_OPERAND (pfn0, 0))) I don't like duplicating the logic for when we might know the pfn is non-null; that seems fragile. I'd rather go with the tf_no

Re: Fix PR64876, regressions in powerpc64 Go testsuite

2015-02-03 Thread Jakub Jelinek
On Wed, Feb 04, 2015 at 12:27:35AM +1030, Alan Modra wrote: > +static bool > +chain_already_loaded (rtx_insn *last) > +{ > + if (last != NULL) > +{ > + rtx patt = PATTERN (last); > + > + if (GET_CODE (patt) == SET) > + { > + rtx lhs = XEXP (patt, 0); > + > + if (REG_P

Fix PR64876, regressions in powerpc64 Go testsuite

2015-02-03 Thread Alan Modra
This fixes a large number of Go testsuite failures on powerpc64 ELFv1, caused by loading r11 from a function descriptor and thus trashing the value set up from CALL_EXPR_STATIC_CHAIN. So don't load r11 if it already contains a useful value. Whether r11 has been set is found directly by examining

Re: [patch] Fix invalid attributes in libstdc++

2015-02-03 Thread Renlin Li
On 01/02/15 15:08, Jonathan Wakely wrote: I failed to CC gcc-patches on this patch ... On 29/01/15 13:02 +, Jonathan Wakely wrote: diff --git a/libstdc++-v3/testsuite/17_intro/headers/c++200x/all_attributes.cc b/libstdc++-v3/testsuite/17_intro/headers/c++200x/all_attributes.cc new file mod

Re: [PATCH/AARCH64] Fix 64893: ICE with vget_lane_u32 with C++ front-end at -O0

2015-02-03 Thread pinskia
> On Feb 3, 2015, at 3:57 AM, Alan Lawrence wrote: > > > Andrew Pinski wrote: >> While trying to build the GCC 5 with GCC 5, I ran into an ICE when >> building libcpp at -O0. The problem is the C++ front-end was not >> folding sizeof(a)/sizeof(a[0]) when passed to a function at -O0. The >>

Re: Merge current set of OpenACC changes from gomp-4_0-branch

2015-02-03 Thread Julian Brown
On Tue, 3 Feb 2015 14:28:44 +0300 Ilya Verbin wrote: > Hi Julian! > > On 27 Jan 14:07, Julian Brown wrote: > > On Mon, 26 Jan 2015 17:34:26 +0300 > > Ilya Verbin wrote: > > > Here is my current patch, it works for OpenMP->MIC, but obviously > > > will not work for PTX, since it requires symmetr

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

2015-02-03 Thread Alex Velenko
On 03/02/15 08:29, Bin.Cheng wrote: On Tue, Feb 3, 2015 at 3:24 PM, Jeff Law wrote: On 02/02/15 08:59, Alex Velenko wrote: On 11/10/14 13:44, Felix Yang wrote: Hello Jeff, I see that you have improved the RTL typesafety issue for ira.c, so I rebased this patch on the latest tru

Re: [RFC][PR target/39726 P4 regression] match.pd pattern to do type narrowing

2015-02-03 Thread Joseph Myers
On Tue, 3 Feb 2015, Jeff Law wrote: > +/* Given a bit-wise operation performed in mode P1 on operands > + in some narrower type P2 that feeds an outer masking operation. > + See if the mask turns off all the bits outside P2, and if so > + perform the all the operations in P2 and just convert

Re: [PATCH/AARCH64] Fix 64893: ICE with vget_lane_u32 with C++ front-end at -O0

2015-02-03 Thread Alan Lawrence
Andrew Pinski wrote: While trying to build the GCC 5 with GCC 5, I ran into an ICE when building libcpp at -O0. The problem is the C++ front-end was not folding sizeof(a)/sizeof(a[0]) when passed to a function at -O0. The C++ front-end keeps around sizeof until the gimplifier and there is no wa

Re: [RFC][PR target/39726 P4 regression] match.pd pattern to do type narrowing

2015-02-03 Thread Richard Biener
On February 2, 2015 7:32:15 PM CET, Jeff Law wrote: >On 02/02/15 01:57, Richard Biener wrote: >>> >>> The nice thing about wrapping the result inside a convert is the >types for >>> the inner operations will propagate from the type of the inner >operands, >>> which is exactly what we want. We the

Re: Merge current set of OpenACC changes from gomp-4_0-branch

2015-02-03 Thread Ilya Verbin
Hi Julian! On 27 Jan 14:07, Julian Brown wrote: > On Mon, 26 Jan 2015 17:34:26 +0300 > Ilya Verbin wrote: > > Here is my current patch, it works for OpenMP->MIC, but obviously > > will not work for PTX, since it requires symmetrical changes in the > > plugin. Could you please take a look, whethe

Re: [PATCH][libstdc++][Testsuite] isctype test fails for newlib.

2015-02-03 Thread Paolo Carlini
Hi, On 02/03/2015 11:40 AM, Matthew Wahab wrote: Ok to commit? Ok thanks. Paolo.

[C++ Patch/RFC] PR 64877

2015-02-03 Thread Paolo Carlini
Hi, Manuel did most of the work on this [5 Regression], caused by my fix for c++/43906, which extended a lot the functionality of -Waddress: a spurious warning is emitted with -Waddress for an expression internally generated in cp_build_binary_op. Manuel suggested that when safe we could comp

Re: [PATCH] Fix CSE volatile MEM handling (PR rtl-optimization/64756)

2015-02-03 Thread Eric Botcazou
> 2015-02-02 Jakub Jelinek > > PR rtl-optimization/64756 > * cse.c (cse_insn): If dest != SET_DEST (sets[i].rtl) and > HASH (SET_DEST (sets[i].rtl), mode) computation sets do_not_record, > invalidate and do not record it. > > * gcc.c-torture/execute/pr64756.c: New

Re: [PATCH][libstdc++][Testsuite] isctype test fails for newlib.

2015-02-03 Thread Matthew Wahab
On 03/02/15 10:27, Paolo Carlini wrote: Nit: the path should be * testsuite/28_regex/... and likewise for the other testcase, because it starts where is the corresponding ChangeLog. Fixed changelog: libstdc++-v3/ 2015-02-02 Matthew Wahab PR libstdc++/64467 * testsuite/28_re

Re: [patch] Fix invalid attributes in libstdc++

2015-02-03 Thread Iain Sandoe
Hi Jonathan, On 1 Feb 2015, at 15:10, Jonathan Wakely wrote: > On 01/02/15 15:08 +, Jonathan Wakely wrote: >> I failed to CC gcc-patches on this patch ... >> >> On 29/01/15 13:02 +, Jonathan Wakely wrote: >>> Jakub pointed out that we have some attributes that don't use the >>> reserved

Re: [PATCH][libstdc++][Testsuite] isctype test fails for newlib.

2015-02-03 Thread Paolo Carlini
Hi, On 02/03/2015 11:17 AM, Matthew Wahab wrote: libstdc++-v3/testsuite/ 2015-02-02 Matthew Wahab PR libstdc++/64467 * 28_regex/traits/char/isctype.cc (test01): Add newlib special case for '\n'. * 28_regex/traits/wchar_t/isctype.cc (test01): Likewise. Nit: the path should be

Re: [PATCH][libstdc++][Testsuite] isctype test fails for newlib.

2015-02-03 Thread Matthew Wahab
[Email problems so resending to the list, sorry for multiple copies.] On 02/02/15 16:33, Jonathan Wakely wrote: On 2 February 2015 at 16:17, Paolo Carlini wrote: This is https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64467 so please note that in the ChangeLog. I guess the patch is Ok for trunk

[SPARC] Adjust multiply costs in 64-bit mode

2015-02-03 Thread Eric Botcazou
As discovered by Rainer, an oversight in sparc_rtx_costs causes the function to return very high costs for multiply operations with -m64 for default V9 or new processors (when TARGET_DEPRECATED_V8_INSNS is not set to be precise). Fixed by the attached patch to config/sparc/sparc.c, the config/sp

Re: [PATCH] Fix combiner from accessing or writing out of bounds SET_N_REGS (PR other/63504)

2015-02-03 Thread Eric Botcazou
> Changed in my copy to > /* One plus the highest pseudo for which we track REG_N_SETS. */ > > Ok with that change? OK if you also add (part of) the explanation you wrote down in the opening message to the above comment. -- Eric Botcazou

[PATCH]Remve xfail for wrapped target from libstdc++-v3/testsuite/27_io/ios_base/sync_with_stdio/1.cc

2015-02-03 Thread Renlin Li
Hi all, This patch simply remove the target selector. It should pass for all target which applies. The comment in the code is not correct. stderr is redirected, not the stdout. Therefore, the return status which is streamed into stdout should properly captured even by wrapped target. Okay f

Re: [PATCH] i386: XFAIL the scan-assembler in pr49095.c (PR61225)

2015-02-03 Thread Jakub Jelinek
On Tue, Feb 03, 2015 at 12:49:19AM -0800, Segher Boessenkool wrote: > As discussed in PR61225, we won't be able to fix the minor regression here > for GCC 5, so let's XFAIL this test for ia32. > > Tested on x86_64-linux -m32 and -m64. Okay for mainline? > > > Segher > > > 2015-02-03 Segher B

[PATCH] i386: XFAIL the scan-assembler in pr49095.c (PR61225)

2015-02-03 Thread Segher Boessenkool
As discussed in PR61225, we won't be able to fix the minor regression here for GCC 5, so let's XFAIL this test for ia32. Tested on x86_64-linux -m32 and -m64. Okay for mainline? Segher 2015-02-03 Segher Boessenkool gcc/testsuite/ PR middle-end/61225 gcc.target/i386/pr49095

Re: [PATCH, v0] fortran: !GCC$ unroll for DO

2015-02-03 Thread Tobias Burnus
Bernhard Reutner-Fischer wrote: > Some compilers IIRC use !DIR$ unroll, if memory serves me right then > the DEC compiler had !DEC$ unroll. > We could support one or the other three-letter keyword or maybe not. Intel's compiler supports quite a lot of loop directives. (Its Fortran front end is bas

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

2015-02-03 Thread Bin.Cheng
On Tue, Feb 3, 2015 at 3:24 PM, Jeff Law wrote: > On 02/02/15 08:59, Alex Velenko wrote: >> >> On 11/10/14 13:44, Felix Yang wrote: >>> >>> Hello Jeff, >>> >>> I see that you have improved the RTL typesafety issue for ira.c, >>> so I rebased this patch >>> on the latest trunk and change