Re: [C PATCH] c_incomplete_type_error TLC

2015-04-29 Thread Jeff Law
On 04/29/2015 12:51 PM, Marek Polacek wrote: This patch cleans up c_incomplete_type_error a bit: today, we should prefer using %qT over %<%s %E%> (the code is largely intact since 1992), and second, we should print the type if we can. The "invalid use of incomplete typedef" error wasn't tested a

RE: [PATCH v4][MIPS] fix CRT_CALL_STATIC_FUNCTION macro

2015-04-29 Thread Moore, Catherine
> -Original Message- > From: Petar Jovanovic [mailto:petar.jovano...@rt-rk.com] > Sent: Friday, April 24, 2015 10:51 AM > Subject: [PATCH v4][MIPS] fix CRT_CALL_STATIC_FUNCTION macro > > gcc/ChangeLog: > > 2015-04-21 Petar Jovanovic > > * config/mips/mips.h (CRT_CALL_STATIC_FU

Re: [PATCH] Refactor handle_section_attribute to reduce nesting and distinguish error cases

2015-04-29 Thread Jeff Law
On 04/26/2015 07:09 PM, Josh Triplett wrote: handle_section_attribute contains many levels of nested conditionals and branching code flow paths, with the error cases sometimes in the else case and sometimes in the if case. Simplify the code flow into a series of potential failure cases ending wi

Re: C++ PATCH for c++/50800 (ICE with may_alias and templates)

2015-04-29 Thread Jason Merrill
On 04/29/2015 07:41 AM, Markus Trippelsdorf wrote: I'm getting hundreds of new warnings when building LLVM, e.g.: Whoops. Fixed thus. commit 7da5a68ba55cd49c46ff9ac547df59248bd90f7d Author: Jason Merrill Date: Wed Apr 29 16:04:08 2015 -0400 PR c++/50800 * tree.c (apply_identity

RE: [Patch, MIPS] Change mips4 default processor to r10K

2015-04-29 Thread Moore, Catherine
> -Original Message- > From: Matthew Fortune [mailto:matthew.fort...@imgtec.com] > Sent: Tuesday, April 28, 2015 2:38 PM > > Steve Ellcey writes: > > > > 2015-04-28 Steve Ellcey > > > > * config/mips/mips-cpus.def: (mips4): Change default processor > > from PROCESSOR_R8000 to

Re: [PATCH, i386]: Fix PR 65871, bzhi builtin/intrinsic wrongly assumes bzhi instruction doesn't set the ZF flag

2015-04-29 Thread Uros Bizjak
On Wed, Apr 29, 2015 at 9:02 PM, Uros Bizjak wrote: > Hello! > > Attached patch implements a CCZ-only setting pattern for a couple of > BMI[,2] intrinsics. These two insns are three operand instructions, and some peephole2 patterns were not prepared for this. Please note that clobbered register c

Re: [PATCH 5/14][AArch64] Add basic fp16 support

2015-04-29 Thread Joseph Myers
On Wed, 22 Apr 2015, Alan Lawrence wrote: > [Resending with correct in-reply-to header] > > This adds basic support for moving __fp16 values around, passing and > returning, and operating on them by promoting to 32-bit floats. Also a few > scalar testcases. I'd think it would be desirable to sha

Re: [PATCH] Fix size & type for cold partition names (hot-cold function partitioning)

2015-04-29 Thread Caroline Tice
Here is a new patch to update the cold name partition so that it will only be treated like a function name and be given a size on the architectures that specifically define macros for such. I also updated the test case to try to only test on the appropriate architectures. I am not sure I got the

New Danish PO file for 'gcc' (version 5.1.0)

2015-04-29 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 Danish team of translators. The file is available at: http://translationproject.org/latest/gcc/da.po (This file, 'gcc-5.1.0.da.po', has just

Re: [PATCH 3/8] add default for PCC_BITFIELD_TYPE_MATTERS

2015-04-29 Thread Trevor Saunders
On Wed, Apr 29, 2015 at 02:35:39PM +0200, Andreas Schwab wrote: > spawn /daten/aranym/gcc/gcc-20150429/Build/gcc/xgcc > -B/daten/aranym/gcc/gcc-20150429/Build/gcc/ > /daten/aranym/gcc/gcc-20150429/gcc/testsuite/objc/execute/bf-1.m > -fno-diagnostics-show-caret -fdiagnostics-color=

libgo patch committed: Support -buildmode=c-archive

2015-04-29 Thread Ian Lance Taylor
The upcoming Go 1.5 release of the gc compiler adds support for "go build -buildmode=c-archive". This can be used to build Go code into an archive (a .a file) that can be linked with a non-Go program, such that the non-Go program can call into Go code. This patch adds support for that to gccgo.

Re: [PATCH] gfortran testsuite: implicitly cleanup-modules

2015-04-29 Thread Joseph Myers
On Thu, 23 Apr 2015, Bernhard Reutner-Fischer wrote: > I've done the above some time ago, dejagnu >= 1.5.2 has a "libdirs" > variable so i'd like to propose to: > > - Bump the required dejagnu version for gcc-6 to (let's say) 1.5.3 1.5.2 is only a few months old. I suggest waiting until a given

Re: [PATCH 3/8] add default for PCC_BITFIELD_TYPE_MATTERS

2015-04-29 Thread Andreas Schwab
Trevor Saunders writes: > it passes for me on x86_64-linux after that commit, what platform is > this? Any one with #undef PCC_BITFIELD_TYPE_MATTERS. See libobjc/encoding.c. Andreas. -- Andreas Schwab, sch...@linux-m68k.org GPG Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276

Re: [Patch/rtl-expand] Take tree range info into account to improve LSHIFT_EXP expanding

2015-04-29 Thread Jiong Wang
Jeff Law writes: > On 04/27/2015 02:21 PM, Jiong Wang wrote: > >> Jeff, >> >>Sorry, I can't understand the meaning of "overlap between t_low and low", >>assume "right" in "right value" means the opposite of "left" not >>"correct". >> >>So what you mean is t_low and low share the s

Re: [PATCH 3/8] add default for PCC_BITFIELD_TYPE_MATTERS

2015-04-29 Thread Jeff Law
On 04/29/2015 03:55 PM, Andreas Schwab wrote: Trevor Saunders writes: it passes for me on x86_64-linux after that commit, what platform is this? Any one with #undef PCC_BITFIELD_TYPE_MATTERS. See libobjc/encoding.c. Can't you just answer the question Andreas instead of making Trevor go loo

Go patch committed: Propagate escape info from closures to enclosed variables

2015-04-29 Thread Ian Lance Taylor
This patch from Chris Manghane fixes the Go frontend to propagate escape information from closures to the enclosed variables to which they refer. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian diff -r 0010ef165479 go/escape.cc --- a/go/escape.cc We

Re: [PATCH 3/8] add default for PCC_BITFIELD_TYPE_MATTERS

2015-04-29 Thread Trevor Saunders
On Wed, Apr 29, 2015 at 04:00:47PM -0600, Jeff Law wrote: > On 04/29/2015 03:55 PM, Andreas Schwab wrote: > >Trevor Saunders writes: > > > >>it passes for me on x86_64-linux after that commit, what platform is > >>this? > > > >Any one with #undef PCC_BITFIELD_TYPE_MATTERS. See libobjc/encoding.c.

Re: [PATCH 3/8] add default for PCC_BITFIELD_TYPE_MATTERS

2015-04-29 Thread Jeff Law
On 04/29/2015 04:25 PM, Trevor Saunders wrote: On Wed, Apr 29, 2015 at 04:00:47PM -0600, Jeff Law wrote: On 04/29/2015 03:55 PM, Andreas Schwab wrote: Trevor Saunders writes: it passes for me on x86_64-linux after that commit, what platform is this? Any one with #undef PCC_BITFIELD_TYPE_MA

Re: [PATCH 3/8] add default for PCC_BITFIELD_TYPE_MATTERS

2015-04-29 Thread Jeff Law
On 04/29/2015 04:25 PM, Trevor Saunders wrote: On Wed, Apr 29, 2015 at 04:00:47PM -0600, Jeff Law wrote: On 04/29/2015 03:55 PM, Andreas Schwab wrote: Trevor Saunders writes: it passes for me on x86_64-linux after that commit, what platform is this? Any one with #undef PCC_BITFIELD_TYPE_MA

Re: [Patch/rtl-expand] Take tree range info into account to improve LSHIFT_EXP expanding

2015-04-29 Thread Jeff Law
On 04/29/2015 03:36 PM, Jiong Wang wrote: Jeff Law writes: On 04/27/2015 02:21 PM, Jiong Wang wrote: Jeff, Sorry, I can't understand the meaning of "overlap between t_low and low", assume "right" in "right value" means the opposite of "left" not "correct". So what you mean

Go patch committed: Fix escape analysis for multi-result calls

2015-04-29 Thread Ian Lance Taylor
This patch from Chris Manghane fixes the handling of multi-result calls in escape analysis. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian diff -r 76359c093067 go/escape.cc --- a/go/escape.cc Wed Apr 29 15:09:15 2015 -0700 +++ b/go/escape.cc W

Re: [C/C++ PATCH] Implement -Wshift-negative-value (PR c/65179)

2015-04-29 Thread Joseph Myers
On Mon, 27 Apr 2015, Marek Polacek wrote: > trigger by default. One change is that we reject programs that use shift with > undefined behavior in a context where a constant expression is required, thus > e.g. enum E { A = -1 << 0 }; > But I hope that's reasonable. That seems appropriate (for C99

Re: [PATCH 0/13] Add musl support to GCC

2015-04-29 Thread Joseph Myers
On Wed, 29 Apr 2015, Szabolcs Nagy wrote: > only affects [u]int_fastN_t types > (on 64bit systems for N=16,32 musl uses int but glibc uses long) > > i can fix glibc-stdint.h, but it's yet another way in which the > compiler is tied to a particular libc. > > (using musl-stdint.h would be nicer, b

[C++ Patch] PR 61683

2015-04-29 Thread Paolo Carlini
Hi, this seems pretty straightforward given the grammar. Tested x86_64-linux. Thanks, Paolo. /cp 2015-04-30 Paolo Carlini PR c++/61683 * parser.c (cp_parser_mem_initializer): Allow for decltype-specifier. /testsuite 2015-04-30 Paolo Carlini

Re: [PATCH 3/8] add default for PCC_BITFIELD_TYPE_MATTERS

2015-04-29 Thread Trevor Saunders
On Wed, Apr 29, 2015 at 04:29:26PM -0600, Jeff Law wrote: > On 04/29/2015 04:25 PM, Trevor Saunders wrote: > >On Wed, Apr 29, 2015 at 04:00:47PM -0600, Jeff Law wrote: > >>On 04/29/2015 03:55 PM, Andreas Schwab wrote: > >>>Trevor Saunders writes: > >>> > it passes for me on x86_64-linux after

[debug-early] fix -fdump-go-spec

2015-04-29 Thread Aldy Hernandez
Hi guys! Despite what Go thinks: /* The debug hooks are used to implement -fdump-go-spec because it gives a simple and stable API for all the information we need to dump. */ ...the debug hooks are not stable... :). The godump-1.c test is failing in the debug-early branch. It seem

mismatch in configure.ac for libgomp

2015-04-29 Thread Pedro Giffuni
Hello; I was checking out some old gcc revisions and I found this commit: https://gcc.gnu.org/viewcvs/gcc?view=revision&revision=140497 It has an obvious error that is still carried today: Instead of adding adding the check for the header a redundant check for was added. The mismatch is sti

Re: [PATCH 3/8] add default for PCC_BITFIELD_TYPE_MATTERS

2015-04-29 Thread Trevor Saunders
On Wed, Apr 29, 2015 at 08:13:16PM -0400, Trevor Saunders wrote: > On Wed, Apr 29, 2015 at 04:29:26PM -0600, Jeff Law wrote: > > On 04/29/2015 04:25 PM, Trevor Saunders wrote: > > >On Wed, Apr 29, 2015 at 04:00:47PM -0600, Jeff Law wrote: > > >>On 04/29/2015 03:55 PM, Andreas Schwab wrote: > > >>>T

Re: [PATCH] Fix PR62283

2015-04-29 Thread H.J. Lu
On Tue, Apr 28, 2015 at 1:29 AM, Richard Biener wrote: > > The following fixes a missed optimization in basic-block vectorization. > Currently we require the SLP chain to end up in a sequence of loads > we support. But of course we can in theory end the SLP chain at > any point and simply constru

Re: [PATCH] add self-tuning to x86 hardware fast path in libitm

2015-04-29 Thread Nuno Diegues
Hello, I have taken the chance to improve the patch by addressing the comments above in this thread. Namely: - to use a simple random generator managed inside the library only - removed floating point usage and replaced by fixed arithmetic - added some comments where relevant Re-running the ST

Re: [PATCH, RFC]: Next stage1, refactoring: propagating rtx subclasses

2015-04-29 Thread Jeff Law
On 04/29/2015 01:55 AM, Mikhail Maltsev wrote: I probably would have done separate patches for the std::swap changes. They're not really related to the rtx subclasses work. OK, sending 2 separate patches. Note that they a not "commutative": std::swap should be applied before the main one, beca

More type narrowing in match.pd

2015-04-29 Thread Jeff Law
This is an incremental improvement to the type narrowing in match.pd. It's largely based on the pattern I added to fix 47477. Basically if we have (bit_and (arith_op (convert A) (convert B)) mask) Where the conversions are widening and the mask turns off all bits outside the original types

Re: [PATCH 3/8] add default for PCC_BITFIELD_TYPE_MATTERS

2015-04-29 Thread Jeff Law
On 04/29/2015 08:10 PM, Trevor Saunders wrote: I decided to commit this, it seems like testing it can be slow on some targets and I did a bootstrap on x86_64-linux-gnu (with regtest queued) and it seems very very unlikely to break anything else. Seems reasonable.Thanks for taking care of it

Re: [PATCH] Fix size & type for cold partition names (hot-cold function partitioning)

2015-04-29 Thread Jack Howarth
The new patch bootstraps fine on x86_64-apple-darwin14. On Wed, Apr 29, 2015 at 5:22 PM, Caroline Tice wrote: > Here is a new patch to update the cold name partition so that it will > only be treated like a function name and be given a size on the > architectures that specifically define macros f

Re: [patch] Rewrite check_global_declarations() generically

2015-04-29 Thread Aldy Hernandez
On 04/29/2015 12:33 PM, Jason Merrill wrote: On 04/28/2015 09:01 PM, Aldy Hernandez wrote: The approach looks good to me. - analyze_functions (); + analyze_functions (true); In the C++ front end at least we comment anonymous boolean arguments, i.e. analyze_functions (/*first_time*/true)

Re: [PATCH] add self-tuning to x86 hardware fast path in libitm

2015-04-29 Thread Andi Kleen
Nuno Diegues writes: > Hello, > > I have taken the chance to improve the patch by addressing the > comments above in this thread. > Namely: > - to use a simple random generator managed inside the library only > - removed floating point usage and replaced by fixed arithmetic > - added some comm

Re: [PATCH, RFC]: Next stage1, refactoring: propagating rtx subclasses

2015-04-29 Thread Jeff Law
On 04/29/2015 01:55 AM, Mikhail Maltsev wrote: [ Big Snip ] Couple minor issues. Can you please check the changes to do_jump_1, the indention looked weird in the patch. If it's correct, just say so. The ChangeLog needed some work. I'm attaching the one I'd use for the patch as it stands to

Re: [PATCH] PR target/48904 x86_64-knetbsd-gnu missing defs

2015-04-29 Thread Jeff Law
On 04/29/2015 02:01 AM, Bernhard Reutner-Fischer wrote: 2012-09-21 H.J. Lu PR target/48904 * config.gcc (x86_64-*-knetbsd*-gnu): Add i386/knetbsd-gnu64.h. * config/i386/knetbsd-gnu64.h: New file OK. Please install on the trunk. THanks, Jeff

Re: Refactor gcc/tree-vectorize.c:vectorize_loops

2015-04-29 Thread Jeff Law
On 04/29/2015 08:37 AM, Aditya K wrote: Thanks for the feedback. I have added comment and properly indented the code. I made a couple more formatting fixes (spaces -> tab & line wrapping), improved the ChangeLog, did a bootstrap & regression test on x86_64-linux-gnu and installed the final pat

Re: [PATCH] Fix size & type for cold partition names (hot-cold function partitioning)

2015-04-29 Thread Uros Bizjak
On Wed, Apr 29, 2015 at 11:22 PM, Caroline Tice wrote: > Here is a new patch to update the cold name partition so that it will > only be treated like a function name and be given a size on the > architectures that specifically define macros for such. > > I also updated the test case to try to only

Re: Mostly rewrite genrecog

2015-04-29 Thread Bin.Cheng
On Mon, Apr 27, 2015 at 6:20 PM, Richard Sandiford wrote: > I think it's been the case for a while that parallel builds of GCC tend > to serialise around the compilation of insn-recog.c, especially with > higher --enable-checking settings. This patch tries to speed that > up by replacing most of

Re: [PATCH 3/8] add default for PCC_BITFIELD_TYPE_MATTERS

2015-04-29 Thread Andreas Schwab
Trevor Saunders writes: > actually pointing out libojc/encoding.c was more useful since that makes > it pretty clear the ifndef PCC_BITFIELD_TYPE_MATTERS there just needs to > be changed to #if ! That probably won't work on arm or powerpc or vax: gcc/config/arm/arm.h:#define PCC_BITFIELD_TYPE_M

Re: [PATCH 3/8] add default for PCC_BITFIELD_TYPE_MATTERS

2015-04-29 Thread Andreas Schwab
Trevor Saunders writes: >> diff --git a/libobjc/encoding.c b/libobjc/encoding.c >> index 7333908..20ace46 100644 >> --- a/libobjc/encoding.c >> +++ b/libobjc/encoding.c >> @@ -1167,7 +1167,7 @@ objc_layout_structure_next_member (struct >> objc_struct_layout *layout) >>/* Record must have at

Re: Mostly rewrite genrecog

2015-04-29 Thread Andreas Schwab
Richard Sandiford writes: > /* Represents a test and the action that should be taken on the result. >If a transition exists for the test outcome, the machine switches >to the transition's target state. If no suitable transition exists, >the machine either falls through to the next de

<    1   2