Re: [PATCH] palignr improvement (PR target/62128)

2014-10-02 Thread Jakub Jelinek
On Thu, Oct 02, 2014 at 11:48:27AM +0200, Jakub Jelinek wrote: > Tested with > GCC_TEST_RUN_EXPENSIVE=1 make -k check-gcc \ > RUNTESTFLAGS='--target_board=unix/-mavx2 dg-torture.exp=vshuf*.c' > on x86_64-linux, ok for trunk if it passes bootstrap? > > As for the previous testcase with distilled pr

[PING][PATCH] Support for BIT_FIELD_REF in asan.c

2014-10-02 Thread Marat Zakirov
On 09/26/2014 12:55 PM, Marat Zakirov wrote: Hi all! Here's a patch which instruments byte-aligned BIT_FIELD_REFs. During GCC asan-bootstrap and Linux kernel build I didn't find any cases where BIT_FIELD_REF is not 8 bits aligned. But I do not have sufficient confidence to replace current re

Re: RFA: one more version of the patch for PR61360

2014-10-02 Thread Uros Bizjak
On Thu, Oct 2, 2014 at 10:13 PM, Vladimir Makarov wrote: I guess we achieved the consensus about the following patch to fix PR61360 https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61360 The patch was successfully bootstrapped and tested (w/wo -march=amdfam10) on x8

libgo patch committed: Check CPU_COUNT, not glibc version

2014-10-02 Thread Ian Lance Taylor
This patch to libgo, based on one from Bryan Hundven, fixes libgo's use of CPU_COUNT to check whether CPU_COUNT is defined, rather than checking the glibc version. This works with other libraries, like uclibc. This is issue 38 in the gofrontend issue tracker. Bootstrapped and ran Go testsuite on

std::is_final for gcc-4.9 branch.

2014-10-02 Thread Ed Smith-Rowland
Built and tested clean on x86_64-linux. OK? Ed 2014-10-03 Edward Smith-Rowland <3dw...@verizon.net> * include/std/type_traits: Add is_final<> type trait for C++14. * testsuite/util/testsuite_tr1.h: Add * testsuite/20_util/is_final/requirements/ explicit_insta

Add dynamic flag into polymorphic call context

2014-10-02 Thread Jan Hubicka
Hi, this patch adds very basic logic for tracking dynamic wrt static types. Basically it intoruced field DYNAMIC that specify if context represent static storage or type found by get_dynamic_type. Static storage is stronger in a way that we know its size (I do not track at all for dynamic types) a

[PATCH] Fix dupplicate declaration of ggc_realloc in gencondmd PR63429

2014-10-02 Thread tsaunders
From: Trevor Saunders Hi, If vec.h is included before ggc.h it forward declares ggc_realloc with defaulted arguments. This means ggc.h can not be included later because it would lead to a second declaration of ggc_realloc with defaulted arguments. In generator programs vec.h can not include gg

Re: [debug-early] C++: emit early debug info for all globals, not just statics

2014-10-02 Thread Aldy Hernandez
On 10/02/14 12:56, Richard Biener wrote: Yes, but I don't see why we need the cleanup step. The decls are real after all. You won't hear me complain. I was just worried people were going to start complaining about larger .debug_info sections, since I see that the current code, as it stands i

Re: [PATCH] SPARC: add mcpu=leon3v7 target

2014-10-02 Thread Eric Botcazou
[Sorry for the long delay] > The LEON3/4 soft-core CPU has support for both SPARCv7 and SPARCv8 that > is configurable at design time. The majority of the LEON3 ASICs are v8 > compatible, however when designing an as small LEON3 as possible, v7 > without FPU is frequently used. > > The current GC

Re: [PATCH C++] - SD-6 Implementation Part 3 - .

2014-10-02 Thread Paolo Carlini
Hi, On 10/02/2014 10:08 AM, Ed Smith-Rowland wrote: On 10/02/2014 02:51 AM, Ed Smith-Rowland wrote: On 10/01/2014 11:28 AM, Jonathan Wakely wrote: On 02/09/14 10:24 +0100, Jonathan Wakely wrote: On 01/09/14 21:46 -0400, Ed Smith-Rowland wrote: Index: include/bits/stl_function.h =

Re: RFA: one more version of the patch for PR61360

2014-10-02 Thread Vladimir Makarov
On 2014-10-02 7:31 AM, Markus Trippelsdorf wrote: On 2014.10.02 at 09:17 +0200, Markus Trippelsdorf wrote: On 2014.09.26 at 16:31 -0400, Vladimir Makarov wrote: I guess we achieved the consensus about the following patch to fix PR61360 https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61360 The pa

Re: [PATCH] Avoid ICE in LRA for calls with multiple return values

2014-10-02 Thread Ilya Enkovich
2014-10-02 22:30 GMT+04:00 Jeff Law : > On 10/02/14 08:30, Ilya Enkovich wrote: >> >> Hi, >> >> This patch adds a check for call destination register for a call return >> value optimization based on REG_RETURNED note. This solves some ICE issues >> for MPX codes. >> >> Bootstrapped and tested on l

Re: [debug-early] C++: emit early debug info for all globals, not just statics

2014-10-02 Thread Richard Biener
On October 2, 2014 8:21:51 PM CEST, Aldy Hernandez wrote: >On 10/02/14 08:46, Jason Merrill wrote: >> On 10/01/2014 05:31 PM, Aldy Hernandez wrote: >>> + for (tree t = level->names; t; t = TREE_CHAIN(t)) >>> +if (TREE_CODE (t) != TYPE_DECL >>> +&& TREE_CODE (t) != PARM_DECL >>> +&& !D

Re: [PATCH] Add libstdc++ baseline_symbols for aarch64

2014-10-02 Thread Christophe Lyon
Hello, On 29 September 2014 11:56, Jonathan Wakely wrote: > On 26/09/14 23:42 +0200, Andreas Schwab wrote: >> >> Generated by make new-abi-baseline on aarch64-suse-linux. >> >> Andreas. >> >> * config/abi/post/aarch64-linux-gnu/baseline_symbols.txt: New >> file. > > > OK, thanks.

Re: [debug-early] do not add location info/etc to abstract instances

2014-10-02 Thread Aldy Hernandez
On 10/02/14 08:53, Jason Merrill wrote: On 09/30/2014 02:43 PM, Aldy Hernandez wrote: + if (parm_die + /* Make sure the function to which this parameter belongs to is + not an abstract instance. If it is, we can't reuse anything. + We must create a new DW_TAG_formal_parameter with

Re: [PATCH, rs6000] Generate LE code for vec_lvsl and vec_lvsr that is compatible with BE code

2014-10-02 Thread Bill Schmidt
Hi, Here's a revised version of the patch that addresses Segher's comments. Bootstrapped and tested on powerpc64le-unknown-linux-gnu. Ok for trunk? Thanks, Bill [gcc] 2014-10-02 Bill Schmidt * altivec.md (altivec_lvsl): New define_expand. (altivec_lvsl_direct): Rename defi

Re: __intN patch 3/5: main __int128 -> __intN conversion.

2014-10-02 Thread DJ Delorie
> > The test would only pass for msp430x (and fail for msp430, which is > > the same target back-end). Do I need to redo the big patch, or would > > a separate one suffice? > > Separate is fine. Turns out it's mangled like this: __int20 foo (__int20 a, unsigned __int20 b); _Z3foou5int20u6uint

Re: parallel check output changes?

2014-10-02 Thread Andrew MacLeod
On 10/02/2014 02:14 PM, Segher Boessenkool wrote: On Thu, Oct 02, 2014 at 06:46:19PM +0100, Richard Sandiford wrote: Segher Boessenkool writes: On Wed, Sep 24, 2014 at 10:54:57AM -0400, Andrew MacLeod wrote: Is this suppose to be resolved now? I'm still seeing some issues with a branch cut f

Re: [PATCH] Fix PR63422 to handle profile insanities

2014-10-02 Thread Jeff Law
On 10/01/14 11:03, Xinliang David Li wrote: Probably need to file a bug to track the copyrename2 problem. Yea, that would definitely be appreciated. Jeff

Re: [PATCH] Fix PR63422 to handle profile insanities

2014-10-02 Thread Jeff Law
On 10/01/14 10:59, Teresa Johnson wrote: This patch removes some asserts my jump threading patch r215739 added. An upstream pass (copyrename2) is introducing some bogus profile counts, so we can't assert that counts are 0 when there is no profile data for the function. Tested on testcase attache

[RFC, RFH PATCH, i386] Fix gcc.target/i386/pr61403.c FAIL with -mavx2

2014-10-02 Thread Uros Bizjak
On Wed, Oct 1, 2014 at 9:03 PM, Uros Bizjak wrote: >> And now the expand_vec_perm_palignr improvement, tested >> with GCC_TEST_RUN_EXPENSIVE=1 make check-gcc \ >> RUNTESTFLAGS='--target_board=unix/-mavx2 dg-torture.exp=vshuf*.c' >> E.g. >> typedef unsigned long long V __attribute__ ((vector_size

Re: [PATCH] Redesign jump threading profile updates

2014-10-02 Thread Jeff Law
On 10/01/14 14:04, Teresa Johnson wrote: The block frequencies are very small in this case leading to rounding errors when computing the edge frequency. The rounding error was then propagated into the recomputed probabilities, leading to insanities in the outgoing edge probabilities on the jump t

Re: Fix hard_regno bitfield in IRA allocno struct

2014-10-02 Thread Jeff Law
On 10/02/14 04:29, David Sherwood wrote: Hi, At the request of Andreas Schwab I have fixed the hard_regno bitfield in the allocno structure as signed integers need to be explicitly marked as signed in bitfields. Cheers, Dave. gcc/ChangeLog: 2014-10-02 David Sherwood * ira-int.h:

Re: [patch] Do not generate useless integral conversions

2014-10-02 Thread Jeff Law
On 10/02/14 04:40, Eric Botcazou wrote: [This is an old discussion about useless integral conversions introduced behind the back of the front-end by the routines in convert.c] I don't like re-introducing that inconsistency. OK. Maybe instead make convert.c do if (!TYPE_UNSIGNED) unsigned_ty

Re: [PATCH] Avoid ICE in LRA for calls with multiple return values

2014-10-02 Thread Jeff Law
On 10/02/14 08:30, Ilya Enkovich wrote: Hi, This patch adds a check for call destination register for a call return value optimization based on REG_RETURNED note. This solves some ICE issues for MPX codes. Bootstrapped and tested on linux-x86_64. OK for trunk? Thanks, Ilya -- 2014-10-02 I

Re: [debug-early] C++: emit early debug info for all globals, not just statics

2014-10-02 Thread Aldy Hernandez
On 10/02/14 08:46, Jason Merrill wrote: On 10/01/2014 05:31 PM, Aldy Hernandez wrote: + for (tree t = level->names; t; t = TREE_CHAIN(t)) +if (TREE_CODE (t) != TYPE_DECL +&& TREE_CODE (t) != PARM_DECL +&& !DECL_IS_BUILTIN (t)) + debug_hooks->early_global_decl (t); What does t

Re: [C++] Handle && || ! for simd vectors

2014-10-02 Thread Jason Merrill
OK. Sorry for the delay, please feel free to ping patches as often as once a week. Jason

Re: parallel check output changes?

2014-10-02 Thread Segher Boessenkool
On Thu, Oct 02, 2014 at 06:46:19PM +0100, Richard Sandiford wrote: > Segher Boessenkool writes: > > On Wed, Sep 24, 2014 at 10:54:57AM -0400, Andrew MacLeod wrote: > >> Is this suppose to be resolved now? I'm still seeing some issues with a > >> branch cut from mainline from yesterday. > > > > C

Re: [C++ Patch] Add default arguments to cp_parser_assignment_expression and cp_parser_constant_expression

2014-10-02 Thread Paolo Carlini
Hi, On 10/02/2014 08:03 PM, Jason Merrill wrote: On 08/19/2014 08:18 AM, Paolo Carlini wrote: - /*non_constant_p=*/&dummy); + &dummy); Why remove the comment? Oh well, the rationale was that normally we use that sort of comment only to explain integer litera

Re: [PATCH] gcc.dg/vect/ cleanup

2014-10-02 Thread Marek Polacek
On Thu, Oct 02, 2014 at 11:58:37AM -0600, Jeff Law wrote: > In fact, I think I'll just go ahead and pre-approve the other patches of > this nature. Thanks. I have a bunch of similar patches that just need proper CL entries, but they're of the same nature: missing declarations, defaulting to int,

Re: RFA: RTL typesafety improvements for ira.c

2014-10-02 Thread Jeff Law
On 10/01/14 17:27, David Malcolm wrote: On Wed, 2014-10-01 at 16:34 -0600, Jeff Law wrote: This was inspired by a discussion with Felix who was making changes in this area. Basically this promotes the "init_insns" field within struct equivalence from an rtx to an rtx_insn_list. The only thing

Re: [C++ Patch] Add default arguments to cp_parser_assignment_expression and cp_parser_constant_expression

2014-10-02 Thread Jason Merrill
On 08/19/2014 08:18 AM, Paolo Carlini wrote: - /*non_constant_p=*/&dummy); + &dummy); Why remove the comment? The rest of the patch is OK. Jason

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

2014-10-02 Thread Paolo Carlini
Hi, On 10/02/2014 07:37 PM, Jason Merrill wrote: On 10/02/2014 12:44 PM, Paolo Carlini wrote: + s->noexcept_operand = scope_chain ? cp_noexcept_operand : 0; s->x_stmt_tree.stmts_are_full_exprs_p = true; scope_chain = s; @@ -6182,6 +6183,7 @@ pop_from_top_level_1 (void) current_fun

Re: [PATCH 3/5] Add test cases for all the new cilk errors

2014-10-02 Thread Jeff Law
On 10/01/14 22:26, Andi Kleen wrote: From: Andi Kleen gcc/testsuite/: 2014-09-30 Andi Kleen * c-c++-common/cilk-plus/CK/errors.c: New test. OK. Jeff

Re: [PATCH 4/5] Fix some of the existing Cilk tests for the new errors.

2014-10-02 Thread Jeff Law
On 10/01/14 22:26, Andi Kleen wrote: From: Andi Kleen gcc/testsuite/: 2014-09-30 Andi Kleen * c-c++-common/cilk-plus/AN/misc.c (main): Handle new cilk errors. OK. Jeff

Re: [PATCH 1/5] Fix error location for cilk error message

2014-10-02 Thread Jeff Law
On 10/01/14 22:26, Andi Kleen wrote: From: Andi Kleen Output the correct location for an existing cilk error message. gcc/c-family/: 2014-09-28 Andi Kleen * cilk.c (recognize_spawn): Use expression location for error message.' OK. Jeff

Re: __intN patch 3/5: main __int128 -> __intN conversion.

2014-10-02 Thread DJ Delorie
> Ah, good point. In which case I don't see what this code is trying to > accomplish relative to falling through to the "prefer the unsigned one" > code below. Shall we just remove it? I don't know for sure. There was __int128 code there, I replaced it with the "same" code, so as to avoid an

Re: parallel check output changes?

2014-10-02 Thread Jakub Jelinek
On Thu, Oct 02, 2014 at 06:46:19PM +0100, Richard Sandiford wrote: > Segher Boessenkool writes: > > On Wed, Sep 24, 2014 at 10:54:57AM -0400, Andrew MacLeod wrote: > >> Is this suppose to be resolved now? I'm still seeing some issues with a > >> branch cut from mainline from yesterday. > > > > C

Re: Go patch committed: Fix symbol name adjustments to match go tool

2014-10-02 Thread Ian Lance Taylor
On Thu, Oct 2, 2014 at 10:56 AM, Ian Lance Taylor wrote: > In the go tool, when converting the pkgpath argument to use for a symbol > name, everything except alphanumeric characters is converted to an > underscore character. This is PR 61880. This patch from Alexander > Shopov changes the Go fro

Re: [PATCH] gcc.dg/vect/ cleanup

2014-10-02 Thread Jeff Law
On 10/02/14 07:39, Marek Polacek wrote: This patch is a cleanup of tests in gcc.dg/vect/ directory. See https://gcc.gnu.org/ml/gcc-patches/2014-09/msg02656.html for more info. Tested on x86_64-linux: vanilla results == results with this patch == results with this patch and gnu11 as a default. O

Re: [PATCH, i386, Pointer Bounds Checker 38/x] Avoid warning for missed IPA_REF_CHKP in switch

2014-10-02 Thread Jeff Law
On 10/02/14 07:49, Ilya Enkovich wrote: On 02 Oct 15:34, Marek Polacek wrote: On Thu, Oct 02, 2014 at 05:29:15PM +0400, Ilya Enkovich wrote: Hi, Seems some new warnings appeared lately and now I see a bootstrap failure for mpx branch due to missing IPA_REF_CHKP case in switch statement in pr

Go patch committed: Fix symbol name adjustments to match go tool

2014-10-02 Thread Ian Lance Taylor
In the go tool, when converting the pkgpath argument to use for a symbol name, everything except alphanumeric characters is converted to an underscore character. This is PR 61880. This patch from Alexander Shopov changes the Go frontend to match, so that the symbol names generated by the cgo tool

Re: parallel check output changes?

2014-10-02 Thread Richard Sandiford
Segher Boessenkool writes: > On Wed, Sep 24, 2014 at 10:54:57AM -0400, Andrew MacLeod wrote: >> Is this suppose to be resolved now? I'm still seeing some issues with a >> branch cut from mainline from yesterday. > > Confirmed. The following patch works for me, and Andrew has tested it > as well

Re: __intN patch 3/5: main __int128 -> __intN conversion.

2014-10-02 Thread Jason Merrill
On 10/02/2014 12:41 PM, DJ Delorie wrote: The C++ standard says that extended integer types participate in the usual arithmetic conversions. If I add a 32-bit int and an __int48, the usual arithmetic conversions should convert the int to __int48. Except the code you're referring to isn't part

Re: [PATCH] gcc.dg/noncompile/ cleanup

2014-10-02 Thread Jeff Law
On 10/02/14 10:48, Marek Polacek wrote: This patch is a cleanup of tests in gcc.dg/noncompile/ directory. See https://gcc.gnu.org/ml/gcc-patches/2014-09/msg02656.html for more info. Tested on x86_64-linux: vanilla results == results with this patch == results with this patch and gnu11 as a defau

Re: [PATCH] gcc.dg/tree-prof/ cleanup

2014-10-02 Thread Jeff Law
On 10/02/14 11:01, Marek Polacek wrote: This patch is a cleanup of tests in gcc.dg/tree-prof/ directory. See https://gcc.gnu.org/ml/gcc-patches/2014-09/msg02656.html for more info. Tested on x86_64-linux: vanilla results == results with this patch == results with this patch and gnu11 as a defaul

Re: [PATCH] Improve prepare_shrink_wrap to sink more instructions

2014-10-02 Thread Jiong Wang
On 02/10/14 09:49, Andrew Pinski wrote: On Fri, Sep 19, 2014 at 1:43 PM, Jeff Law wrote: On 09/15/14 08:33, Jiong Wang wrote: Jeff, thanks, I partially understand your meaning here. take the function "ira_implicitly_set_insn_hard_regs" in ira-lives.c for example, when generatin

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

2014-10-02 Thread Jason Merrill
On 10/02/2014 12:44 PM, Paolo Carlini wrote: + s->noexcept_operand = scope_chain ? cp_noexcept_operand : 0; s->x_stmt_tree.stmts_are_full_exprs_p = true; scope_chain = s; @@ -6182,6 +6183,7 @@ pop_from_top_level_1 (void) current_function_decl = s->function_decl; cp_unevaluated_o

Re: feature-testing macros for 4.9

2014-10-02 Thread Thiago Macieira
On Thursday 02 October 2014 11:42:35 Jonathan Wakely wrote: > The library macros are safe and can't cause any issues, so I'm happy for > them to go on the branch. > > I can think of at least two that aren't implemented on the branch: > and the std::is_final trait (although I'd be OK with the > tr

Re: [PATCH] Provide global var location info for asan

2014-10-02 Thread Jakub Jelinek
On Wed, Sep 24, 2014 at 12:38:05PM +0200, Jakub Jelinek wrote: > Ok for trunk? > > 2014-09-24 Jakub Jelinek > > * ubsan.h (ubsan_get_source_location): New prototype. > * ubsan.c (ubsan_source_location_type): New variable. > Function renamed to ... > (ubsan_get_source_lo

[PATCHv5] Vimrc config with GNU formatting

2014-10-02 Thread Yury Gribov
On 09/17/2014 09:08 PM, Yury Gribov wrote: > On 09/16/2014 08:38 PM, Yury Gribov wrote: >> Hi all, >> >> This is the third version of the patch. A list of changes since last >> version: >> * move config to contrib so that it's _not_ enabled by default (current >> score is 2/1 in favor of no Vim co

[jit] Merger of trunk r215803 into jit branch

2014-10-02 Thread David Malcolm
I've merged the last ~3 weeks of changes of svn trunk to the git branch dmalcolm/jit, pushing it as dbfbacdbbf727e8ab3a8cff2d5650d8000de0449. Specifically, this is from trunk r215803 aka 68874c5d021dd6b8b740ffd23960e46e333270b5 (2014-10-02) bringing in all changes since r215090 aka65d8ffc23985

Re: parallel check output changes?

2014-10-02 Thread Jakub Jelinek
On Thu, Oct 02, 2014 at 11:47:39AM -0500, Segher Boessenkool wrote: > On Wed, Sep 24, 2014 at 10:54:57AM -0400, Andrew MacLeod wrote: > > Is this suppose to be resolved now? I'm still seeing some issues with a > > branch cut from mainline from yesterday. > > Confirmed. The following patch works

[PATCH] gcc.dg/tree-prof/ cleanup

2014-10-02 Thread Marek Polacek
This patch is a cleanup of tests in gcc.dg/tree-prof/ directory. See https://gcc.gnu.org/ml/gcc-patches/2014-09/msg02656.html for more info. Tested on x86_64-linux: vanilla results == results with this patch == results with this patch and gnu11 as a default. Ok? 2014-10-02 Marek Polacek

[PATCH] gcc.dg/noncompile/ cleanup

2014-10-02 Thread Marek Polacek
This patch is a cleanup of tests in gcc.dg/noncompile/ directory. See https://gcc.gnu.org/ml/gcc-patches/2014-09/msg02656.html for more info. Tested on x86_64-linux: vanilla results == results with this patch == results with this patch and gnu11 as a default. Ok? 2014-10-02 Marek Polacek

Re: parallel check output changes?

2014-10-02 Thread Segher Boessenkool
On Wed, Sep 24, 2014 at 10:54:57AM -0400, Andrew MacLeod wrote: > Is this suppose to be resolved now? I'm still seeing some issues with a > branch cut from mainline from yesterday. Confirmed. The following patch works for me, and Andrew has tested it as well. The comment it removes isn't valid

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

2014-10-02 Thread Paolo Carlini
Hi, On 10/02/2014 04:26 PM, Jason Merrill wrote: On 10/02/2014 08:11 AM, Paolo Carlini wrote: +/* Nonzero if we are parsing the operand of a noexcept operator. */ + +extern int cp_noexcept_operand; Rather than add a global variable, let's look it up in scope_chain directly, like current_nam

Re: __intN patch 3/5: main __int128 -> __intN conversion.

2014-10-02 Thread DJ Delorie
> The C++ standard says that extended integer types participate in the > usual arithmetic conversions. If I add a 32-bit int and an __int48, the > usual arithmetic conversions should convert the int to __int48. Except the code you're referring to isn't part of that conversion. It only handles

[jit] Implement documentation targets within jit/Make-lang.in

2014-10-02 Thread David Malcolm
On Tue, 2014-09-23 at 23:27 +, Joseph S. Myers wrote: [...] > Shouldn't the jit.pdf, jit.install-html etc. Make-lang.in hooks actually > build / install the documentation for this JIT? [...] I've committed the following to the branch dmalcolm/jit: This patch implements the documentation targ

Re: [PATCH] Fix PR63422 to handle profile insanities

2014-10-02 Thread Teresa Johnson
On Wed, Oct 1, 2014 at 10:03 AM, Xinliang David Li wrote: > Probably need to file a bug to track the copyrename2 problem. I have filed https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63443 to track this. > > David > > On Wed, Oct 1, 2014 at 9:59 AM, Teresa Johnson wrote: >> This patch removes some

Re: [PATCH] Redesign jump threading profile updates

2014-10-02 Thread Teresa Johnson
On Thu, Oct 2, 2014 at 8:45 AM, Steve Ellcey wrote: > On Wed, 2014-10-01 at 22:02 -0700, Teresa Johnson wrote: > >> Will take a look. In case I can't reproduce it with the aarch64 >> cross-compiler or x86_64, can you give me the preprocessed source? And >> also the instructions for how to configur

Re: [debug-early] do not add location info/etc to abstract instances

2014-10-02 Thread Jason Merrill
On 10/02/2014 11:53 AM, Jason Merrill wrote: On 09/30/2014 02:43 PM, Aldy Hernandez wrote: + if (parm_die + /* Make sure the function to which this parameter belongs to is + not an abstract instance. If it is, we can't reuse anything. + We must create a new DW_TAG_formal_parameter

Re: [debug-early] do not add location info/etc to abstract instances

2014-10-02 Thread Jason Merrill
On 09/30/2014 02:43 PM, Aldy Hernandez wrote: + if (parm_die + /* Make sure the function to which this parameter belongs to is +not an abstract instance. If it is, we can't reuse anything. +We must create a new DW_TAG_formal_parameter with a +corresponding DW_AT_abs

Re: Why doesn't include ?

2014-10-02 Thread Jonathan Wakely
On 17/09/14 16:55 +0200, Marc Glisse wrote: On Wed, 17 Sep 2014, Jonathan Wakely wrote: Our only includes , not as required by 26.8 [c.math] paragraph 1. Is that intentional? Probably not (I don't know), but if you change it, please include instead of the nonsensical (and equivalent) . I

Re: [debug-early] C++: emit early debug info for all globals, not just statics

2014-10-02 Thread Jason Merrill
On 10/01/2014 05:31 PM, Aldy Hernandez wrote: + for (tree t = level->names; t; t = TREE_CHAIN(t)) +if (TREE_CODE (t) != TYPE_DECL + && TREE_CODE (t) != PARM_DECL + && !DECL_IS_BUILTIN (t)) + debug_hooks->early_global_decl (t); What does this do for templates? I think we d

Re: [libgo, arm64] Future of reflection

2014-10-02 Thread Richard Henderson
On 10/01/2014 03:08 PM, Ian Lance Taylor wrote: >> func TestMakeFunc(t *testing.T) { >> switch runtime.GOARCH { >> case "amd64", "386": >> default: >> t.Skip("MakeFunc not implemented for " + runtime.GOARCH) >> } > > Wait, what sources are you looki

Re: [PATCH] Redesign jump threading profile updates

2014-10-02 Thread Steve Ellcey
On Wed, 2014-10-01 at 22:02 -0700, Teresa Johnson wrote: > Will take a look. In case I can't reproduce it with the aarch64 > cross-compiler or x86_64, can you give me the preprocessed source? And > also the instructions for how to configure for MIPS. > > Teresa Never mind, I messed up the patch

Re: [PATCH] Redesign jump threading profile updates

2014-10-02 Thread Teresa Johnson
On Wed, Oct 1, 2014 at 10:02 PM, Teresa Johnson wrote: > On Wed, Oct 1, 2014 at 3:46 PM, Steve Ellcey wrote: >> On Wed, 2014-10-01 at 13:04 -0700, Teresa Johnson wrote: >> >>> 2014-10-01 Teresa Johnson >>> >>> * tree-ssa-threadupdate.c (freqs_to_counts_path): Scale frequencies >>>

Re: feature-testing macros for 4.9

2014-10-02 Thread Jonathan Wakely
The library macros are safe and can't cause any issues, so I'm happy for them to go on the branch. I can think of at least two that aren't implemented on the branch: and the std::is_final trait (although I'd be OK with the trait going on the branch too). - Original Message - From: "

Re: feature-testing macros for 4.9

2014-10-02 Thread Jason Merrill
On 10/02/2014 10:58 AM, Ed Smith-Rowland wrote: We've had a request from someone that we include SD-6 feature testing for gcc-4.9. It seems rather heavy for a backport but it might make maintainers of large C++ libraries happy ;-) and it might make sense as people migrate from 4.9 to 5.0 to see w

[PATCH 4/n] OpenMP 4.0 offloading infrastructure: lto-wrapper

2014-10-02 Thread Ilya Verbin
Hello, With this patch lto-wrapper performs invocation of mkoffload tool for each offload target. This tool should be provided by the corresponding offload compiler. It will compile IR from .gnu.offload_lto_* sections into offload target code and embed the resultant code (offload image) into the

Re: [PATCH 2/5] Error out for Cilk_spawn or array expression in forbidden places

2014-10-02 Thread Joseph S. Myers
On Wed, 1 Oct 2014, Andi Kleen wrote: > +/* Check that no array notation or spawn statement is in EXPR. > + If not true gemerate an error at LOC for WHAT. */ > + > +bool > +check_no_cilk (tree expr, const char *what, location_t loc) > +{ > + if (!flag_cilkplus) > +return false; > + if (co

Re: [PATCH 0/10] OpenACC 2.0 support for libgomp

2014-10-02 Thread Julian Brown
Hi, On Wed, 24 Sep 2014 14:32:31 +0200 Jakub Jelinek wrote: > On Tue, Sep 23, 2014 at 07:17:25PM +0100, Julian Brown wrote: > > The upcoming patch series constitutes our current (still > > in-progress) implementation of run-time support for OpenACC 2.0 in > > libgomp. We've tried to build on top

Re: __intN patch 3/5: main __int128 -> __intN conversion.

2014-10-02 Thread Jason Merrill
On 10/02/2014 10:22 AM, Jason Merrill wrote: Extended integer types larger than long long also participate in enumeral promotion if needed, but I think your changes to c_common_type_for_size handle that. ...and it seems we currently don't support enums with underlying type larger than long lon

Re: [PATCH,i386] Fix adxintrin on mingw.

2014-10-02 Thread H.J. Lu
On Thu, Oct 2, 2014 at 7:29 AM, Ilya Tocar wrote: > Hi, > > sizeof (long) == 4 on windows, so we should use long long as param type. > Patch below does it. The same is true for x32. Can you add a testcase to show it fails on x32 without the fix? > Ok for trunk? > > 2014-10-02 Ilya Tocar > >

[PATCH] Avoid ICE in LRA for calls with multiple return values

2014-10-02 Thread Ilya Enkovich
Hi, This patch adds a check for call destination register for a call return value optimization based on REG_RETURNED note. This solves some ICE issues for MPX codes. Bootstrapped and tested on linux-x86_64. OK for trunk? Thanks, Ilya -- 2014-10-02 Ilya Enkovich * lra-constraints.

[PATCH,i386] Fix adxintrin on mingw.

2014-10-02 Thread Ilya Tocar
Hi, sizeof (long) == 4 on windows, so we should use long long as param type. Patch below does it. Ok for trunk? 2014-10-02 Ilya Tocar * config/i386/adxintrin.h (_subborrow_u64): Use long long for param type. (_addcarry_u64): Ditto. (_addcarryx_u64): Ditto. ---

Re: [PATCH, i386, Pointer Bounds Checker 33/x] MPX ABI

2014-10-02 Thread Ilya Enkovich
On 01 Oct 14:57, Vladimir Makarov wrote: > On 2014-09-25 5:46 AM, Ilya Enkovich wrote: > >2014-09-25 1:51 GMT+04:00 Ilya Enkovich : > >>2014-09-24 23:09 GMT+04:00 Jeff Law : > >>>On 09/24/14 07:13, Ilya Enkovich wrote: > > I tried to generate PARALLEL with all regs set by call. Here is a

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

2014-10-02 Thread Jason Merrill
On 10/02/2014 08:11 AM, Paolo Carlini wrote: +/* Nonzero if we are parsing the operand of a noexcept operator. */ + +extern int cp_noexcept_operand; Rather than add a global variable, let's look it up in scope_chain directly, like current_namespace and such. Jason

Re: [wwwdocs] Add feature-testing macros and std::is_final to gcc-5/changes.html

2014-10-02 Thread Jonathan Wakely
On 02/10/14 10:09 -0400, Ed Smith-Rowland wrote: On 10/02/2014 06:14 AM, Jonathan Wakely wrote: On 02/10/14 11:12 +0100, Jonathan Wakely wrote: Note Ed's recent changes. Committed to CVS. And fix a markup error that I expected xmllint to catch :-( Thank you! I tried to do this and couldn't fo

Re: __intN patch 3/5: main __int128 -> __intN conversion.

2014-10-02 Thread Jason Merrill
On 10/01/2014 11:52 PM, DJ Delorie wrote: It seems like the int128 code here was broken and this is continuing that brokenness. Extended integer types have integer conversion rank corresponding to their bitsize, so int128 should have higher rank than long long, but here it was being checked afte

Re: [wwwdocs] Add feature-testing macros and std::is_final to gcc-5/changes.html

2014-10-02 Thread Ed Smith-Rowland
On 10/02/2014 06:14 AM, Jonathan Wakely wrote: On 02/10/14 11:12 +0100, Jonathan Wakely wrote: Note Ed's recent changes. Committed to CVS. And fix a markup error that I expected xmllint to catch :-( Thank you! I tried to do this and couldn't for permissions. I'm probably not doing it right.

Re: [PATCH, Pointer Bounds Checker 19/x] Support bounds in expand

2014-10-02 Thread Ilya Enkovich
On 24 Sep 12:29, Ilya Enkovich wrote: > 2014-09-24 0:58 GMT+04:00 Jeff Law : > > On 06/05/14 08:46, Ilya Enkovich wrote: > >> > >> 2014-06-05 Ilya Enkovich > >> > >> * calls.c: Include tree-chkp.h, rtl-chkp.h, bitmap.h. > >> (arg_data): Add fields special_slot, pointer_arg and >

Re: [PATCH, i386, Pointer Bounds Checker 30/x] Size relocation

2014-10-02 Thread Ilya Enkovich
On 02 Oct 11:49, Uros Bizjak wrote: > On Thu, Oct 2, 2014 at 10:23 AM, Ilya Enkovich wrote: > > > 2014-10-01 Ilya Enkovich > > > > * config/i386/i386.md (UNSPEC_SIZEOF): New. > > (move_size_reloc_): New. > > * config/i386/predicates.md (symbol_operand): New. > >

Re: [PATCH, i386, Pointer Bounds Checker 38/x] Avoid warning for missed IPA_REF_CHKP in switch

2014-10-02 Thread Ilya Enkovich
On 02 Oct 15:34, Marek Polacek wrote: > On Thu, Oct 02, 2014 at 05:29:15PM +0400, Ilya Enkovich wrote: > > Hi, > > > > Seems some new warnings appeared lately and now I see a bootstrap failure > > for mpx branch due to missing IPA_REF_CHKP case in switch statement in > > process_references funct

[PATCH] gcc.dg/vect/ cleanup

2014-10-02 Thread Marek Polacek
This patch is a cleanup of tests in gcc.dg/vect/ directory. See https://gcc.gnu.org/ml/gcc-patches/2014-09/msg02656.html for more info. Tested on x86_64-linux: vanilla results == results with this patch == results with this patch and gnu11 as a default. Ok? 2014-10-02 Marek Polacek *

[PING][PATCH] Warn about unclosed pragma omp declare target.

2014-10-02 Thread Ilya Tocar
Ping. On 15 Aug 16:26, Ilya Tocar wrote: > Ping. > > On 29 Jul 18:45, Ilya Tocar wrote: > > Hi, > > > > As discussed here in https://gcc.gnu.org/ml/gcc/2014-01/msg00189.html > > Gcc should complain about pragma omp declare target without > > corresponding pragma omp end declare target. This patch

Re: [PATCH, i386, Pointer Bounds Checker 38/x] Avoid warning for missed IPA_REF_CHKP in switch

2014-10-02 Thread Marek Polacek
On Thu, Oct 02, 2014 at 05:29:15PM +0400, Ilya Enkovich wrote: > Hi, > > Seems some new warnings appeared lately and now I see a bootstrap failure for > mpx branch due to missing IPA_REF_CHKP case in switch statement in > process_references function. This function handles varpool_nodes which >

[PATCH, i386, Pointer Bounds Checker 38/x] Avoid warning for missed IPA_REF_CHKP in switch

2014-10-02 Thread Ilya Enkovich
Hi, Seems some new warnings appeared lately and now I see a bootstrap failure for mpx branch due to missing IPA_REF_CHKP case in switch statement in process_references function. This function handles varpool_nodes which cannot have references of this kind. This patch fixes a warning. Thanks,

[PATCH] gcc.dg/tm/ cleanup

2014-10-02 Thread Marek Polacek
This patch is a cleanup of tests in gcc.dg/tm/ directory. See https://gcc.gnu.org/ml/gcc-patches/2014-09/msg02656.html for more info. Tested on x86_64-linux: vanilla results == results with this patch == results with this patch and gnu11 as a default. Ok? 2014-10-02 Marek Polacek * g

[PATCH] gcc.dg/torture/ cleanup

2014-10-02 Thread Marek Polacek
This patch is a cleanup of tests in gcc.dg/torture directory. See https://gcc.gnu.org/ml/gcc-patches/2014-09/msg02656.html for more info. Tested on x86_64-linux: vanilla results == results with this patch == results with this patch and gnu11 as a default. Ok? 2014-10-02 Marek Polacek

Re: [PATCH] gcc.dg/tree-ssa/ cleanup

2014-10-02 Thread Marek Polacek
On Thu, Oct 02, 2014 at 01:34:20PM +0200, Jakub Jelinek wrote: > Wouldn't it be worthwhile to avoid such changes for dozen or two randomly > selected testcases from these and instead add -std=gnu89 for them? > Just making sure we don't get rid of all the tests for non-prototyped call > and/or defau

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

2014-10-02 Thread Paolo Carlini
Hi, On 10/02/2014 05:22 AM, Jason Merrill wrote: On 10/01/2014 12:31 PM, Paolo Carlini wrote: in this issue Daniel argued that the value of a noexcept expression should not depend on constructor elision. I'm open to that, but I don't think it's at all clear in the standard. Ok. Personally, I

Re: [PATCH] Implement -fsanitize=object-size

2014-10-02 Thread Jakub Jelinek
On Thu, Sep 11, 2014 at 07:47:51PM +0200, Marek Polacek wrote: > So, how does this look now? Looks much better. There are some nits I'd change, like: 1) no need not to handle bitfields 2) IMHO it should handle PARM_DECL and RESULT_DECL alongside of VAR_DECL 3) decl_p IMHO should use just DECL_P 4

Re: [PATCH] gcc.dg/tree-ssa/ cleanup

2014-10-02 Thread Jakub Jelinek
On Thu, Oct 02, 2014 at 01:30:21PM +0200, Marek Polacek wrote: > This patch is a cleanup of tests in gcc.dg/tree-ssa/ directory. > It is quite large, but trivial - mostly only missing declarations > and defaulting to int. See > https://gcc.gnu.org/ml/gcc-patches/2014-09/msg02656.html > for more i

Re: RFA: one more version of the patch for PR61360

2014-10-02 Thread Markus Trippelsdorf
On 2014.10.02 at 09:17 +0200, Markus Trippelsdorf wrote: > On 2014.09.26 at 16:31 -0400, Vladimir Makarov wrote: > > I guess we achieved the consensus about the following patch to fix PR61360 > > > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61360 > > > > The patch was successfully bootstrappe

[PATCH] gcc.dg/tree-ssa/ cleanup

2014-10-02 Thread Marek Polacek
This patch is a cleanup of tests in gcc.dg/tree-ssa/ directory. It is quite large, but trivial - mostly only missing declarations and defaulting to int. See https://gcc.gnu.org/ml/gcc-patches/2014-09/msg02656.html for more info. Tested on x86_64-linux: vanilla results == results with this patch

Re: [patch] Do not generate useless integral conversions

2014-10-02 Thread Eric Botcazou
[This is an old discussion about useless integral conversions introduced behind the back of the front-end by the routines in convert.c] > I don't like re-introducing that inconsistency. OK. > Maybe instead make convert.c do if (!TYPE_UNSIGNED) unsigned_type_for () > instead? I notice that all

Fix hard_regno bitfield in IRA allocno struct

2014-10-02 Thread David Sherwood
Hi, At the request of Andreas Schwab I have fixed the hard_regno bitfield in the allocno structure as signed integers need to be explicitly marked as signed in bitfields. Cheers, Dave. gcc/ChangeLog: 2014-10-02 David Sherwood * ira-int.h: (ira_allocno) Make hard_regno explicitly sig

Re: [wwwdocs] Add feature-testing macros and std::is_final to gcc-5/changes.html

2014-10-02 Thread Jonathan Wakely
On 02/10/14 11:12 +0100, Jonathan Wakely wrote: Note Ed's recent changes. Committed to CVS. And fix a markup error that I expected xmllint to catch :-( Index: htdocs/gcc-5/changes.html === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-5/ch

  1   2   >