Re: [PATCH, Pointer Bounds Checker 6/x] New static constructor types

2014-09-15 Thread Ilya Enkovich
Ping 2014-06-05 15:03 GMT+04:00 Ilya Enkovich : > 2014-06-04 17:35 GMT+04:00 Richard Biener : >> On Wed, Jun 4, 2014 at 3:13 PM, Ilya Enkovich wrote: >>> 2014-06-04 13:58 GMT+04:00 Richard Biener : On Wed, Apr 16, 2014 at 2:33 PM, Ilya Enkovich wrote: > Hi, > > This patch

Re: [PATCH, Pointer Bounds Checker 20/x] Follow transparent alias chains

2014-09-15 Thread Ilya Enkovich
Ping 2014-06-03 13:01 GMT+04:00 Richard Biener : > On Mon, Jun 2, 2014 at 5:15 PM, Ilya Enkovich wrote: >> Hi, >> >> In the most case we follow transparent alias chains wne assemble names. But >> in some cases it is not performed. For instrumented functions it is >> critical and following pat

Re: [PATCH, i386, Pointer Bounds Checker 32/x] Pointer Bounds Checker hooks for i386 target

2014-09-15 Thread Ilya Enkovich
Ping 2014-06-11 18:00 GMT+04:00 Ilya Enkovich : > Hi, > > This patch adds i386 target hooks for Pointer Bounds Checker. > > Bootstrapped and tested on linux-x86_64. > > Thanks, > Ilya > -- > gcc/ > > 2014-06-11 Ilya Enkovich > > * config/i386/i386.c: Include tree-iterator.h. > (

Re: [PATCH, i386, Pointer Bounds Checker 34/x] Vararg functions support

2014-09-15 Thread Ilya Enkovich
Ping 2014-06-11 18:06 GMT+04:00 Ilya Enkovich : > Hi, > > This patch introduces initialization of incoming bounds for vararg function > on i386 target. > > Bootstrapped and tested on linux-x86_64. > > Thanks, > Ilya > -- > gcc/ > > 2014-06-11 Ilya Enkovich > > * config/i386/i386.c (ix8

Re: [PATCH, i386, Pointer Bounds Checker 31/x] Pointer Bounds Checker builtins for i386 target

2014-09-15 Thread Ilya Enkovich
Ping 2014-06-11 17:58 GMT+04:00 Ilya Enkovich : > Hi, > > This patch adds i386 target builtins for Pointer Bounds Checker. > > Bootstrapped and tested on linux-x86_64. > > Thanks, > Ilya > -- > gcc/ > > 2014-06-11 Ilya Enkovich > > * config/i386/i386-builtin-types.def (BND): New. >

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

2014-09-15 Thread Ilya Enkovich
Ping 2014-06-11 18:03 GMT+04:00 Ilya Enkovich : > Hi, > > This patch adds MPX ABI support for i386 target. > > Bootstrapped and tested on linux-x86_64. > > Thanks, > Ilya > -- > gcc/ > > 2014-06-11 Ilya Enkovich > > * config/i386/i386.c (ix86_option_override_internal): Do not >

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

2014-09-15 Thread Ilya Enkovich
Ping 2014-06-11 12:56 GMT+04:00 Ilya Enkovich : > Hi, > > This patch adds size relocation support for i386 target. Relocation is used > to compute bounds for static objects with incomplete type. > > Thanks, > Ilya > -- > gcc/ > > 2014-06-11 Ilya Enkovich > > * config/i386/i386.md (UNS

Re: [PATCH, i386, Pointer Bounds Checker 18/x] Expand instrumented builtin function calls

2014-09-15 Thread Ilya Enkovich
Ping 2014-06-06 13:32 GMT+04:00 Ilya Enkovich : > 2014-06-03 12:46 GMT+04:00 Richard Biener : >> On Mon, Jun 2, 2014 at 4:51 PM, Ilya Enkovich wrote: >>> Hi, >>> >>> This patch adds support for normal builtin function calls (target ones are >>> not instrumented). The basic idea of the patch is

Re: [PATCH, Pointer Bounds Checker 25/x] DCE

2014-09-15 Thread Ilya Enkovich
Ping 2014-06-06 11:00 GMT+04:00 Ilya Enkovich : > On 03 Jun 17:27, Ilya Enkovich wrote: >> 2014-06-03 15:56 GMT+04:00 Richard Biener : >> > On Tue, Jun 3, 2014 at 1:36 PM, Ilya Enkovich >> > wrote: >> >> 2014-06-03 13:45 GMT+04:00 Richard Biener : >> >>> On Tue, Jun 3, 2014 at 9:23 AM, Ilya Enko

Re: [PATCH, Pointer Bounds Checker 26/x] CCP

2014-09-15 Thread Ilya Enkovich
Ping 2014-06-06 11:02 GMT+04:00 Ilya Enkovich : > On 03 Jun 11:54, Richard Biener wrote: >> On Tue, Jun 3, 2014 at 9:38 AM, Ilya Enkovich wrote: >> > Hi, >> > >> > This patch allows BUILT_IN_CHKP_BNDRET as a consumer of a result of >> > BUILT_IN_STACK_SAVE call. >> > >> > Bootstrapped and tested

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

2014-09-15 Thread Ilya Enkovich
Ping 2014-06-05 18:46 GMT+04:00 Ilya Enkovich : > On 04 Jun 16:36, Michael Matz wrote: >> Hi, >> >> On Mon, 2 Jun 2014, Ilya Enkovich wrote: >> >> > > There is exactly one place (except for the self-recursive ones) where >> > > you call the new store_expr with a non-null argument for bounds >> > >

Re: [PATCH, Pointer Bounds Checker 22/x] Inline

2014-09-15 Thread Ilya Enkovich
Ping 2014-08-18 19:35 GMT+04:00 Ilya Enkovich : > On 06 Jun 11:59, Ilya Enkovich wrote: >> 2014-06-03 13:07 GMT+04:00 Richard Biener : >> > On Mon, Jun 2, 2014 at 5:56 PM, Ilya Enkovich >> > wrote: >> >> + >> >> + /* If indirect call has following BUILT_IN_CHKP_BNDRET >> >> +call th

Re: [patch i386]: Sibcall tail-call improvement and partial fix PR/60104

2014-09-15 Thread Mike Stump
On Sep 14, 2014, at 5:43 PM, Segher Boessenkool wrote: > On Sun, Sep 14, 2014 at 02:38:45PM -0700, Mike Stump wrote: >> + SIBLING_CALL_P (tmp) = 1; >> + SIBLING_CALL_P (tmp) = 1; > > The second time is to make sure? :-) No, just a last minute cut and paste… I’ll remove it.

Re: Stream ODR types

2014-09-15 Thread Richard Biener
On Fri, 12 Sep 2014, Jan Hubicka wrote: > > > For ODR warnings and TBAA I think i want other types, too. But yep, we > > > need to handle > > > gracefuly component types that does not have names and we could drop > > > names of types > > > and handle them as component types as it seems fit. > >

Re: [PATCH, Pointer Bounds Checker 9/x] Cgraph extension

2014-09-15 Thread Ilya Enkovich
On 03 Sep 13:32, Jeff Law wrote: > On 07/24/14 03:59, Ilya Enkovich wrote: > >-- > >2014-07-24 Ilya Enkovich > > > > * cgraph.h (cgraph_thunk_info): Add add_pointer_bounds_args > > field. > > (cgraph_node): Add instrumented_version, orig_decl and > > instrumentation_clone fields.

Re: [PATCH, Pointer Bounds Checker 24/x] PRE

2014-09-15 Thread Ilya Enkovich
On 03 Sep 13:25, Jeff Law wrote: > On 08/18/14 07:02, Ilya Enkovich wrote: > >On 03 Jun 11:33, Richard Biener wrote: > >>On Tue, Jun 3, 2014 at 9:13 AM, Ilya Enkovich > >>wrote: > >>>Hi, > >>> > >>>This patch preserves CALL_WITH_BOUNDS flag for calls during PRE. > >> > >>Ok. > >> > >>Richard. > >

Re: [PATCH. libstdc++] Use the correct C++14 __cplusplus value (201402L). Added C++1z to the preprocessor.

2014-09-15 Thread Paolo Carlini
Hi again, On 08/26/2014 10:54 AM, Paolo Carlini wrote: Hi, On 08/26/2014 10:52 AM, Jonathan Wakely wrote: That seems like a good idea, but I'm not convinced there's any benefit in Ed's changes to do: -#if __cplusplus > 201103L +#if __cplusplus >= 201402L It seems like unnecessary churn to me

Re: [PATCH 1/2] Always set DECL_NO_INSTRUMENT_FUNCTION_ENTRY_EXIT correctly

2014-09-15 Thread Richard Biener
On Fri, Sep 12, 2014 at 4:32 PM, Andi Kleen wrote: > From: Andi Kleen > > When profiling is disabled force DECL_NO_INSTRUMENT_FUNCTION_ENTRY_EXIT > for each function to one. This information is then preserved > through LTO. > > With this patch for LTO builds -pg needs to be set on both the > LTO

Re: [PATCH 2/2] Add some more test cases for fentry and pg

2014-09-15 Thread Richard Biener
On Fri, Sep 12, 2014 at 4:32 PM, Andi Kleen wrote: > From: Andi Kleen > > Test fentry and no_instrument_function overriding. > > No test cases for the LTO test for now, as the LTO > harness doesn't seem to support different flags for the final > link. Sure it does - via dg-extra-ld-options (you

Re: [debug-early] reuse variable DIEs and fix their context

2014-09-15 Thread Richard Biener
On Fri, Sep 12, 2014 at 7:10 PM, Aldy Hernandez wrote: > On 09/12/14 08:15, Jason Merrill wrote: >> >> On 09/11/2014 08:51 PM, Aldy Hernandez wrote: >>> >>> - timevar_start (TV_PHASE_DEFERRED); >> >> >>> - timevar_stop (TV_PHASE_DEFERRED); >>> - timevar_start (TV_PHASE_OPT_GEN); >> >> >> Why? >

[PATCH][PING] Enable -fsanitize-recover for KASan

2014-09-15 Thread Yury Gribov
On 09/05/2014 10:54 AM, Yury Gribov wrote: Hi all, This patch enables -fsanitize-recover for KASan by default. This causes KASan to continue execution after error in case of inline instrumentation. This feature is needed because - reports during early bootstrap won't even be printed - needed to

[PATCH][Kasan][PING] Allow to override Asan shadow offset from command line

2014-09-15 Thread Yury Gribov
On 09/08/2014 06:29 PM, Yury Gribov wrote: Hi all, Kasan developers has asked for an option to override offset of Asan shadow memory region. This should simplify experimenting with memory layouts on 64-bit architectures. I've bootstrapped and regtested this on x64. Ok to commit? -Y commit 9

Re: [arm][patch] fix arm_neon_ok check on !arm_arch7

2014-09-15 Thread Richard Earnshaw
On 13/09/14 22:39, Andrew Stubbs wrote: > Hi, > > I get a lot of "vect/*" and "neon-*" test failure in my armv5te testing > because the arm_neon_ok test incorrectly detects that NEON is valid on > arm926ej-s. > > It turns out that the reason is that the compiler only disallows NEON > for Thumb

Re: [PATCH, Pointer Bounds Checker 23/x] Function split

2014-09-15 Thread Ilya Enkovich
2014-09-03 23:12 GMT+04:00 Jeff Law : > On 08/18/14 09:55, Ilya Enkovich wrote: >> >> On 04 Jun 01:15, Jeff Law wrote: >>> >>> On 06/03/14 01:10, Ilya Enkovich wrote: Hi, This patch does not allow splitting in case bounds are returned until retutrned bounds are supported.

Re: [PATCHv2] Vimrc config with GNU formatting

2014-09-15 Thread Yury Gribov
On 09/14/2014 02:30 PM, Alexander Monakov wrote: On Thu, 11 Sep 2014, Yury Gribov wrote: Ok, it tooks some time. Basically we want brace symbol to behave differently in two contexts: 1) not add any additional offset when not following control flow operator: void f () { int x; { } } N

Re: (Corrected patch) Simple patch for gcc/testsuite/gcc.target/aarch64/vdup_lane_2.c

2014-09-15 Thread James Greenhalgh
On Fri, Sep 05, 2014 at 03:58:30PM +0100, Marcus Shawcroft wrote: > On 5 September 2014 14:46, David Sherwood wrote: > > Apologies, resent with correct patch not containing ChangeLog changes. > > > > > > Hi, > > > > I have a simple patch for the vdup_lane_2.c test that fixes up the > > force_

Re: [PATCH 2/2] add static typed insn_deleted_p

2014-09-15 Thread Yury Gribov
On 09/12/2014 05:17 AM, Trevor Saunders wrote: On Thu, Sep 11, 2014 at 08:06:02PM -0400, David Malcolm wrote: FWIW, in case it's helpful, I have a script here that I use to help make ChangeLog entries: https://github.com/davidmalcolm/gcc-refactoring-scripts/blob/master/generate-changelog.py (nee

Re: [PATCH v2] Use sscanf instead of atoll in gcov-tool.c

2014-09-15 Thread Richard Biener
On Sun, Sep 14, 2014 at 9:04 PM, John David Anglin wrote: > On 8-Sep-14, at 5:21 AM, Richard Biener wrote: > >> On Sun, Sep 7, 2014 at 7:36 PM, John David Anglin >> wrote: >>> >>> The attached patch fixes bootstrap on hpux which doesn't have the atoll >>> function. >>> >>> Tested on hppa2.0w-hp-h

Re: [arm][patch] fix arm_neon_ok check on !arm_arch7

2014-09-15 Thread Andrew Stubbs
On 15/09/14 10:46, Richard Earnshaw wrote: Hmm, I wonder if arm_override_options should reject neon + (arch < 7). Is this more to your taste? Andrew P.S. arm_override_options was renamed in 2010. 2014-09-15 Andrew Stubbs * gcc/config/arm/arm.c (arm_option_override): Reject -mfpu=neon wh

Re: [patch i386]: Sibcall tail-call improvement and partial fix PR/60104

2014-09-15 Thread Iain Sandoe
Hi Mike, On 15 Sep 2014, at 08:33, Mike Stump wrote: > On Sep 14, 2014, at 5:43 PM, Segher Boessenkool > wrote: >> On Sun, Sep 14, 2014 at 02:38:45PM -0700, Mike Stump wrote: >>> + SIBLING_CALL_P (tmp) = 1; >>> + SIBLING_CALL_P (tmp) = 1; >> >> The second time is to make sure? :-) >

[patch] Rename template parameters in std::regex

2014-09-15 Thread Jonathan Wakely
Fix an over-eager find'n'replace. Tested x86_64-linux, committed to trunk. commit 3e9a05fcd4278438229a6d1515ad0d6d3b17f4a9 Author: Jonathan Wakely Date: Mon Sep 15 11:13:52 2014 +0100 * include/bits/regex.h (basic_regex): Rename _Ch_typeraits template parameters to _Ch_traits. diff

Re: [PATCH. libstdc++] Use the correct C++14 __cplusplus value (201402L). Added C++1z to the preprocessor.

2014-09-15 Thread Ed Smith-Rowland
On 09/15/2014 05:17 AM, Paolo Carlini wrote: Hi again, On 08/26/2014 10:54 AM, Paolo Carlini wrote: Hi, On 08/26/2014 10:52 AM, Jonathan Wakely wrote: That seems like a good idea, but I'm not convinced there's any benefit in Ed's changes to do: -#if __cplusplus > 201103L +#if __cplusplus >=

Re: [PATCH. libstdc++] Use the correct C++14 __cplusplus value (201402L). Added C++1z to the preprocessor.

2014-09-15 Thread Jonathan Wakely
On 15/09/14 07:15 -0400, Ed Smith-Rowland wrote: The new libraries are safely tucked away in experimental behind #if __cplusplus <= 201103L # include #else *** Jonathan: uses: #if __cplusplus < 201103L # include #else and should probably eventually be like the others. My Filesystem impleme

Re: [patch i386]: Sibcall tail-call improvement and partial fix PR/60104

2014-09-15 Thread FX
> Perhaps it would be safer simply to revert that hunk of the original patch > unless/until (1) and (2) above are addressed? Given that the original patch addresses “only” a missed-optimization (and causes ice-on-valid), it makes sense to me. FX

[PATCH][match-and-simplify] Enable simplifying of GIMPLE_CONDs

2014-09-15 Thread Richard Biener
To get desired transforms from tree-ssa-forwprop.c which uses fold_stmt. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2014-09-15 Richard Biener * gimple-fold.c: Include tree-eh.c. (fold_stmt_1): Handle GIMPLE_CONDs for gimple_simplify. * gim

Re: [patch i386]: Sibcall tail-call improvement and partial fix PR/60104

2014-09-15 Thread Dominique Dhumieres
> > Perhaps it would be safer simply to revert that hunk of the original patch > > unless/until (1) and (2) above are addressed? > > Given that the original patch addresses 'only' a missed-optimization > (and causes ice-on-valid), it makes sense to me. (1) Iain already asked the questions more tha

Re: [arm][patch] fix arm_neon_ok check on !arm_arch7

2014-09-15 Thread Richard Earnshaw
On 15/09/14 11:56, Andrew Stubbs wrote: > On 15/09/14 10:46, Richard Earnshaw wrote: >> Hmm, I wonder if arm_override_options should reject neon + (arch < 7). > > Is this more to your taste? > Yep, that's fine. > Andrew > > P.S. arm_override_options was renamed in 2010. I'm getting old :-( R

Re: [PATCH] Improve prepare_shrink_wrap to sink more instructions

2014-09-15 Thread Jiong Wang
On 11/09/14 21:39, Jeff Law wrote: On 09/08/14 07:57, Jiong Wang wrote: Conceptually OK. Some questions/concerns about the implementation. Hi Jeff, thanks very much for your review. It seems to me that what you're trying to describe on the RHS is REG_P || CONSTANT_P yes. and a

Re: [PATCH] PowerPC: Implement TARGET_ATOMIC_ASSIGN_EXPAND_FENV

2014-09-15 Thread Maciej W. Rozycki
On Thu, 4 Sep 2014, Adhemerval Zanella wrote: > > While at it, may I propose another change on top of this? > > > > I've noticed the test case is rather slow, it certainly takes much more > > time than the average one, I've seen elapsed times of well over a minute > > on reasonably fast hardwa

Re: [PATCH] PR62120

2014-09-15 Thread Ilya Tocar
On 01 Sep 18:38, Ilya Tocar wrote: > > Please mention the PR in the ChangeLog entry and add some testcases > > (can be gcc.target/i386/, but we should have it tested). > > Does this change anything on say register short sil __asm ("sil"); in 32-bit > > mode (when it IMHO should be rejected too?)? >

[PATCH] AArch64: Improve regmove_costs for 128-bit types

2014-09-15 Thread Wilco Dijkstra
Hi, This patch improves the register move costs for 128-bit types. OK for commit? ChangeLog: 2014-09-15 Wilco Dijkstra * gcc/config/aarch64/aarch64.c (aarch64_register_move_cost): Add register move costs for 128-bit types. --- gcc/config/aarch64/aarch64.c | 29 ++

Re: [PATCH, Pointer Bounds Checker 26/x] CCP

2014-09-15 Thread Jeff Law
On 09/15/14 01:20, Ilya Enkovich wrote: 2014-06-05 Ilya Enkovich * tree-ssa-ccp.c: Include tree-chkp.h. (insert_clobber_before_stack_restore): Handle BUILT_IN_CHKP_BNDRET calls. diff --git a/gcc/tree-ssa-ccp.c b/gcc/tree-ssa-ccp.c index eeefeaf..e99bcf5 100644 -

Re: [PATCH, 2/2] shrink wrap a function with a single loop: split live_edge

2014-09-15 Thread Jiong Wang
On 08/05/14 09:07, Zhenqiang Chen wrote: static bool move_insn_for_shrink_wrap (basic_block bb, rtx insn, const HARD_REG_SET uses, const HARD_REG_SET defs, - HARD_REG_SET *last_uses) + HA

[PATCH,i386] Properly check xgetbv for zmm support.

2014-09-15 Thread Ilya Tocar
Hi, Currently we don't check zmm/mask-registers related bits in xgetbv output, when detecting native cpu. Patch below fixes it. Bootstraps/passes make check. Ok for trunk? ChangeLog: gcc/ 2014-09-15 Ilya Tocar * config/i386/driver-i386.c (host_detect_local_cpu): Detect lack of

Re: [PATCH] microblaze: microblaze.md: Use 'SI' instead of 'VOID' for operand 1 of 'call_value_intern'

2014-09-15 Thread Michael Eager
On 09/14/14 21:19, Chen Gang wrote: On 9/15/14 11:03, Michael Eager wrote: On 09/14/14 00:51, Chen Gang wrote: Hello maintainers: I also find some warnings during compiling microblaze, I also shall try to fix them, but excuse me, I am not quite familiar the testsuite for microblaze, could you

Re: [PATCH,i386] Properly check xgetbv for zmm support.

2014-09-15 Thread Uros Bizjak
On Mon, Sep 15, 2014 at 5:28 PM, Ilya Tocar wrote: > Currently we don't check zmm/mask-registers related bits in xgetbv > output, when detecting native cpu. Patch below fixes it. > Bootstraps/passes make check. > Ok for trunk? > > ChangeLog: > > gcc/ > 2014-09-15 Ilya Tocar > > * confi

Re: [PATCH, Pointer Bounds Checker 25/x] DCE

2014-09-15 Thread Jeff Law
On 09/15/14 01:20, Ilya Enkovich wrote: Here is a fixed version. Bootstrapped and tested on linux_x86-64. Thanks, Ilya -- gcc/ 2014-06-05 Ilya Enkovich * tree-ssa-dce.c: Include tree-chkp.h. (propagate_necessity): For free call fed by alloc check bounds are al

Re: [patch i386]: Sibcall tail-call improvement and partial fix PR/60104

2014-09-15 Thread Jeff Law
On 09/15/14 05:25, FX wrote: Perhaps it would be safer simply to revert that hunk of the original patch unless/until (1) and (2) above are addressed? Given that the original patch addresses “only” a missed-optimization (and causes ice-on-valid), it makes sense to me. What I think we need is f

Re: [PATCH, Pointer Bounds Checker 23/x] Function split

2014-09-15 Thread Jeff Law
On 09/15/14 03:51, Ilya Enkovich wrote: 2014-08-15 Ilya Enkovich * ipa-split.c: Include tree-chkp.h. (find_retbnd): New. (consider_split): Do not split retbnd and retval producers. (split_function): Propagate Pointer Bounds Checker instrum

Re: [C++14 PATCH] Do not diagnose lambda default arguments in c++14 modes.

2014-09-15 Thread Jason Merrill
On 09/14/2014 10:55 AM, Ville Voutilainen wrote: * g++.dg/cpp0x/lambda/lambda-defarg.C: Enable in c++11_only. * g++.dg/cpp1y/lambda-defarg.C: New. Instead of adding a second test, make only the dg-error c++11_only. Jason

Re: [PATCH, Pointer Bounds Checker 26/x] CCP

2014-09-15 Thread Ilya Enkovich
On 15 Sep 09:20, Jeff Law wrote: > On 09/15/14 01:20, Ilya Enkovich wrote: > > >> > >>2014-06-05 Ilya Enkovich > >> > >> * tree-ssa-ccp.c: Include tree-chkp.h. > >> (insert_clobber_before_stack_restore): Handle > >> BUILT_IN_CHKP_BNDRET calls. > >> > >> > >>diff --git a/

Re: [PATCH 2/2] Add some more test cases for fentry and pg

2014-09-15 Thread Andi Kleen
On Mon, Sep 15, 2014 at 11:29:50AM +0200, Richard Biener wrote: > On Fri, Sep 12, 2014 at 4:32 PM, Andi Kleen wrote: > > From: Andi Kleen > > > > Test fentry and no_instrument_function overriding. > > > > No test cases for the LTO test for now, as the LTO > > harness doesn't seem to support diffe

Re: [patch i386]: Sibcall tail-call improvement and partial fix PR/60104

2014-09-15 Thread Iain Sandoe
Hi Jeff, On 15 Sep 2014, at 16:42, Jeff Law wrote: > On 09/15/14 05:25, FX wrote: >>> Perhaps it would be safer simply to revert that hunk of the original patch >>> unless/until (1) and (2) above are addressed? >> >> Given that the original patch addresses “only” a missed-optimization (and >>

Re: [PATCH] gcc parallel make check

2014-09-15 Thread Jakub Jelinek
On Fri, Sep 12, 2014 at 04:42:25PM -0700, Mike Stump wrote: > On Sep 12, 2014, at 9:32 AM, Jakub Jelinek wrote: > > Here is my latest version of the patch. > > I did a timing test: Here is an updated version. Changes since last version: 1) acats parallelized the same way (just, because it is in

[PATCH] Avoid inter-test dependencies in gfortran.dg (PR fortran/56408)

2014-09-15 Thread Jakub Jelinek
Hi! Here is an attempt to fix gfortran dg.exp testing in case of very fine-grained parallelization. So far tested just with make check-gfortran "RUNTESTFLAGS=dg.exp='binding_label_tests_10_main.f03 binding_label_tests_11_main.f03 binding_label_tests_13_main.f03 binding_label_tests_26b.f90 coarra

Re: [PATCH, Pointer Bounds Checker 23/x] Function split

2014-09-15 Thread Ilya Enkovich
2014-09-15 19:39 GMT+04:00 Jeff Law : > On 09/15/14 03:51, Ilya Enkovich wrote: 2014-08-15 Ilya Enkovich * ipa-split.c: Include tree-chkp.h. (find_retbnd): New. (consider_split): Do not split retbnd and retval producers.

[obvious] Use LangEnabledBy for Wpsabi

2014-09-15 Thread Manuel López-Ibáñez
Bootstrapped and regression tested on x86_64-linux-gnu. Committed as obvious at r215272. gcc/c-family/ChangeLog: 2014-09-15 Manuel López-Ibáñez * c.opt (Wpsabi): Use LangEnabledBy. * c-opts.c (c_common_handle_option): Do not handle here. Index: gcc/c-family/c.opt

Re: [C++14 PATCH] Do not diagnose lambda default arguments in c++14 modes.

2014-09-15 Thread Ville Voutilainen
On 15 September 2014 18:51, Jason Merrill wrote: > On 09/14/2014 10:55 AM, Ville Voutilainen wrote: >> >> * g++.dg/cpp0x/lambda/lambda-defarg.C: Enable in c++11_only. >> * g++.dg/cpp1y/lambda-defarg.C: New. > > > Instead of adding a second test, make only the dg-error c++11_only. Ok.

Re: [PATCH] Add direct support for Linux kernel __fentry__ patching

2014-09-15 Thread Andi Kleen
Andi Kleen writes: Ping! > From: Andi Kleen > > The Linux kernel dynamically patches in __fentry__ calls in and > out at runtime. This allows using function tracing for debugging > in production kernels without (significant) performance penalty. > > For this it needs a table pointing to each __

[PATCH 1/n] OpenMP 4.0 offloading infrastructure

2014-09-15 Thread Ilya Verbin
Hello, This patch contains necessary changes for the build system to support offloading. It adds 2 new options for configure: * --enable-as-accelerator-for=ARG is intended for the offload target compiler. * --enable-offload-targets=LIST is intended for the offload host compiler. Some more info:

[C PATCH] Better diagnostics for C++ comments in C90 (PR c/61854)

2014-09-15 Thread Marek Polacek
PR 61854 points out that using // comments in C90 mode without GNU extensions enabled results in a suboptimal error such as: error: expected expression before ‘/’ token This patch improves this diagnostics, so that the compiler now says "C++ style comments are forbidden in C90". We must be careful

Re: [PATCH, Fortran] Wrong invocation of caf_atomic_op

2014-09-15 Thread Alessandro Fanfarillo
New patch after the update. Cheers 2014-09-09 0:30 GMT-06:00 Tobias Burnus : > Alessandro Fanfarillo wrote: >> >> This email follows the previous without subject (sorry about that). > > > I think I'd prefer the following patch, which avoids a temporary if none is > required. "value" is a pointer

Re: [PATCH] gcc parallel make check

2014-09-15 Thread Mike Stump
On Sep 15, 2014, at 9:05 AM, Jakub Jelinek wrote: All the updates sound good. > Regtested on x86_64-linux, without the patch toplevel make -k check > took 8hrs3minutes (don't have time data for that run), This confuses me, but, no matter. Isn’t 8hrs time data? :-) > patch toplevel make -j48

Re: [PATCH] Avoid inter-test dependencies in gfortran.dg (PR fortran/56408)

2014-09-15 Thread Mike Stump
On Sep 15, 2014, at 9:13 AM, Jakub Jelinek wrote: > Here is an attempt to fix gfortran dg.exp testing in case of very > fine-grained parallelization. Ok.

Re: [C PATCH] Better diagnostics for C++ comments in C90 (PR c/61854)

2014-09-15 Thread Joseph S. Myers
On Mon, 15 Sep 2014, Marek Polacek wrote: > We must be careful to properly handle code such as "1 //**/ 2", which > has a different meaning in C90 and GNU90 mode. New testcases test this. I don't think there's sufficient allowance here for other valid cases. It's valid to have // inside #if 0

Re: [debug-early] reuse variable DIEs and fix their context

2014-09-15 Thread Jason Merrill
On 09/15/2014 05:32 AM, Richard Biener wrote: Btw, if the VTV stuff really needs to come afterwards how does VTV work with LTO then? It's conservatively correct: it emits information for all the vtables that are actually generated for the TU. If LTO optimizes some of them away, the VTV infor

[match-and-simplify] remove dt_node::level_max

2014-09-15 Thread Prathamesh Kulkarni
* genmatch.c (dt_node::level_max): Remove. Thanks, Prathamesh Index: genmatch.c === --- genmatch.c (revision 215271) +++ genmatch.c (working copy) @@ -466,7 +466,6 @@ struct dt_simplify: public dt_node { - static const unsigned l

Re: [PATCH, Fortran] Wrong invocation of caf_atomic_op

2014-09-15 Thread Tobias Burnus
On 15.09.2014 19:18, Alessandro Fanfarillo wrote: New patch after the update. 2014-09-09 0:30 GMT-06:00 Tobias Burnus: >I think I'd prefer the following patch, which avoids a temporary if none is >required. "value" is a pointer if the kind is the same (see kind check >before) and if it is not

[wwwdocs] Buildstat update for 4.9

2014-09-15 Thread Tom G. Christensen
Latest results for 4.9.x There are no new results, this is just a rediff. -tgc Testresults for 4.9.1: aarch64-linux-gnu (cross) arm-unknown-linux-gnu hppa-unknown-linux-gnu i686-unknown-linux-gnu mips-unknown-linux-gnu mipsel-unknown-linux-gnu powerpc-unknown-linux-gnu sparc-unkn

[match-and-simplify] CSE with expression captures

2014-09-15 Thread Prathamesh Kulkarni
This patch attempts to performs CSE on expression captures. I have reused captures[] to store the result of expression (capture::what). * genmatch.c (operand::gen_transform): Add dt_operand ** default argument to operand heirarchy. (expr::gen_transform): Adjust. (capture::gen_transform): Likew

Re: [PATCH, Fortran] Wrong invocation of caf_atomic_op

2014-09-15 Thread Alessandro Fanfarillo
In attachment a test case which fails with the current gcc-trunk version but works when the patch is applied. coarray_35.f90 is my attempt to write a gcc test case. The problem is related with atomic_add. 2014-09-15 12:55 GMT-06:00 Tobias Burnus : > On 15.09.2014 19:18, Alessandro Fanfarillo wrote

Re: Fix pr61848, linux kernel miscompile

2014-09-15 Thread Joseph S. Myers
On Mon, 15 Sep 2014, Alan Modra wrote: > This patch cures the linux kernel boot failure when compiled using > trunk gcc. (Andrew, apologies for hijacking your bugzilla, I started > work on this before finding the bugzilla..) Please include testcases in your patch for each case that you fix. --

[google/gcc-4_9] Fix static var promotion handling for LIPO

2014-09-15 Thread Sharad Singhai
This patch is for the google/gcc-4_9 branch to fix a LIPO issue. This patch addresses handling for static initializer data during static variable promotion in LIPO optimize phase. This is useful for those targets which access initializer data via TOC, such as powerpc. After static var promotion i

Re: [match-and-simplify] CSE with expression captures

2014-09-15 Thread Marc Glisse
On Tue, 16 Sep 2014, Prathamesh Kulkarni wrote: --- gcc/match-builtin.pd(revision 215271) +++ gcc/match-builtin.pd(working copy) @@ -44,8 +44,8 @@ /* ??? There is no way to CSE here. We'd need to support expression captures here, like with (

Re: [google/gcc-4_9] Fix static var promotion handling for LIPO

2014-09-15 Thread Xinliang David Li
ok. David On Mon, Sep 15, 2014 at 1:43 PM, Sharad Singhai wrote: > This patch is for the google/gcc-4_9 branch to fix a LIPO issue. > > This patch addresses handling for static initializer data during > static variable promotion in LIPO optimize phase. This is useful for > those targets which ac

Re: [C PATCH] Better diagnostics for C++ comments in C90 (PR c/61854)

2014-09-15 Thread Marek Polacek
On Mon, Sep 15, 2014 at 05:49:25PM +, Joseph S. Myers wrote: > On Mon, 15 Sep 2014, Marek Polacek wrote: > > > We must be careful to properly handle code such as "1 //**/ 2", which > > has a different meaning in C90 and GNU90 mode. New testcases test this. > > I don't think there's sufficien

PATCH [testsuite]: Update gcc.dg/pr61053.c for x32

2014-09-15 Thread H.J. Lu
Hi, gcc.dg/pr61053.c fails on x32, which has the same alignments for floating point types and the integer types with the same size as x86-64. This patch is needed for x32. Tested on ia32, x32 and x86-64. OK for trunk and 4.8 branch? Thanks. H.J. --- 2014-09-15 H.J. Lu * gcc.dg/pr

Re: [PATCH, Pointer Bounds Checker 23/x] Function split

2014-09-15 Thread Jeff Law
On 09/15/14 10:20, Ilya Enkovich wrote: A problem I'm trying to avoid is that bounds in return statement are not taken into account when checking for data dependencies between parts. It means we may have a case when return statement with bounds is put into split part but bounds producer is not.

Re: [PATCH, Pointer Bounds Checker 26/x] CCP

2014-09-15 Thread Jeff Law
On 09/15/14 09:57, Ilya Enkovich wrote: On 15 Sep 09:20, Jeff Law wrote: On 09/15/14 01:20, Ilya Enkovich wrote: 2014-06-05 Ilya Enkovich * tree-ssa-ccp.c: Include tree-chkp.h. (insert_clobber_before_stack_restore): Handle BUILT_IN_CHKP_BNDRET calls. diff --gi

Re: [PATCH, Pointer Bounds Checker 6/x] New static constructor types

2014-09-15 Thread Jeff Law
On 09/15/14 01:14, Ilya Enkovich wrote: Ping 2014-06-05 15:03 GMT+04:00 Ilya Enkovich : 2014-06-04 17:35 GMT+04:00 Richard Biener : On Wed, Jun 4, 2014 at 3:13 PM, Ilya Enkovich wrote: 2014-06-04 13:58 GMT+04:00 Richard Biener : On Wed, Apr 16, 2014 at 2:33 PM, Ilya Enkovich wrote: Hi, T

Re: [PATCH] Improve prepare_shrink_wrap to sink more instructions

2014-09-15 Thread Jeff Law
On 09/11/14 14:58, Wang Jiong wrote: the reg_found will record the # of registers, and there will be a check to make sure no register or just 1 register in the src. so if "reg_found" is bigger than 1, then src will be assigned to NULL_RTX, that we just stop the move. + if (arg.n

Re: [PATCH, Fortran] Wrong invocation of caf_atomic_op

2014-09-15 Thread Tobias Burnus
On 15.09.2014 22:23, Alessandro Fanfarillo wrote: In attachment a test case which fails with the current gcc-trunk version but works when the patch is applied. coarray_35.f90 is my attempt to write a gcc test case. The problem is related with atomic_add. Well, if it is a "dg-do compile" test, i

Re: [Patch] PR 61692 - Fix for inline asm ICE

2014-09-15 Thread Jeff Law
On 09/14/14 02:13, David Wohlferd wrote: I sent you the file you requested (off list), but never heard back from you about the valgrind results. Just haven't got back to it yet... There's always more to get done on any given day than I have the time for, so I have to prioritize and some things

Re: [PATCH] Fix PR debug/60433: Handle TEMPLATE_TYPE_PARM in dbxout_type

2014-09-15 Thread Jeff Law
On 09/11/14 15:36, John David Anglin wrote: On 9/11/2014 4:18 PM, Jeff Law wrote: On 09/07/14 11:52, John David Anglin wrote: The attached change fixes an ICE in dbxout_type on hppa2.0w-hp-hpux11.11 compiling auto-fn24.C. Tested on hppa2.0w-hp-hpux11.11 and hppa64-hp-hpux11.11. OK for trunk a

Re: [PATCH] microblaze: microblaze.md: Use 'SI' instead of 'VOID' for operand 1 of 'call_value_intern'

2014-09-15 Thread Chen Gang
On 09/15/2014 11:30 PM, Michael Eager wrote: > On 09/14/14 21:19, Chen Gang wrote: >> >> On 9/15/14 11:03, Michael Eager wrote: >>> On 09/14/14 00:51, Chen Gang wrote: Hello maintainers: I also find some warnings during compiling microblaze, I also shall try to fix them, but exc

Re: [PATCH] Avoid inter-test dependencies in gfortran.dg (PR fortran/56408)

2014-09-15 Thread Bernhard Reutner-Fischer
On 15 September 2014 19:49:03 Mike Stump wrote: On Sep 15, 2014, at 9:13 AM, Jakub Jelinek wrote: > Here is an attempt to fix gfortran dg.exp testing in case of very > fine-grained parallelization. Ok. Looks like the right approach, yes. Many thanks for doing this! Cheers, Sent with AquaMa

[gomp4] OpenACC routine directive

2014-09-15 Thread Cesar Philippidis
This patch adds initial support for the OpenACC routine directive. It's not complete just yet because it doesn't implement any of the optional clauses, except for the optional function/subroutine name. As such, it doesn't go beyond marking functions with the "omp declare target" attribute. Accordi

[PATCH] Do not allow non-allocatable registers in scratch_operand

2014-09-15 Thread Segher Boessenkool
Currently, scratch_operand allows all hard registers, also those that cannot be allocated and are only generated explicitly by the backend. This causes problems. Consider the case where combine combines instructions A and B, where B clobbers such a non-allocatable hard reg X, into an instruction

C++ PATCH to template argument abi_tag handling

2014-09-15 Thread Jason Merrill
Jon sent me a test that crashed with an abi_tag on a template argument. We need to propagate tags from template arguments when we first look up the specialization, not wait until instantiation. Tested x86_64-pc-linux-gnu, applying to trunk. commit 44db0cad1ad6b651616ab144896c358274eacf4f Autho

RE: [PATCH 3/3,ARM,libgcc]Code size optimization for the fmul/fdiv and dmul/ddiv function in libgcc

2014-09-15 Thread Tony Wang
Ping? > -Original Message- > From: Tony Wang [mailto:tony.w...@arm.com] > Sent: Thursday, September 04, 2014 10:16 AM > To: 'gcc-patches@gcc.gnu.org' > Cc: Richard Earnshaw; Ramana Radhakrishnan > Subject: RE: [PATCH 3/3,ARM,libgcc]Code size optimization for the fmul/fdiv > and dmul/ddiv

RE: [PATCH 1/3,ARM,libgcc]Code size optimization for the fmul/fdiv and dmul/ddiv function in libgcc

2014-09-15 Thread Tony Wang
Ping? > -Original Message- > From: Tony Wang [mailto:tony.w...@arm.com] > Sent: Thursday, September 04, 2014 10:15 AM > To: 'gcc-patches@gcc.gnu.org' > Cc: Richard Earnshaw; Ramana Radhakrishnan > Subject: RE: [PATCH 1/3,ARM,libgcc]Code size optimization for the fmul/fdiv > and dmul/ddiv

RE: [PATCH 2/3,ARM,libgcc]Code size optimization for the fmul/fdiv and dmul/ddiv function in libgcc

2014-09-15 Thread Tony Wang
Ping? > -Original Message- > From: Tony Wang [mailto:tony.w...@arm.com] > Sent: Thursday, September 04, 2014 10:16 AM > To: 'gcc-patches@gcc.gnu.org' > Cc: Richard Earnshaw; Ramana Radhakrishnan > Subject: RE: [PATCH 2/3,ARM,libgcc]Code size optimization for the fmul/fdiv > and dmul/ddiv

Updated no_reorder patchkit

2014-09-15 Thread Andi Kleen
This version addresses earlier comments and has an updated testsuite (still no LTO tests however). The assembler statements also no stay in order with ordered statements. It doesn't disable sorting of paritions with ordered symbols. I think that's an existing bug and is best addressed separately.

[PATCH 2/2] Add test cases for noreorder

2014-09-15 Thread Andi Kleen
From: Andi Kleen Add some simple test cases for noreorder behaving like -fno-toplevel-reorder and -fno-toplevel-reorder still working. Unfortunately there's no easy way to check for output order in DG, so we just check for existence. gcc/testsuite/: 2014-09-15 Andi Kleen * gcc.dg/no

[PATCH 1/2] Add an no_reorder attribute for LTO

2014-09-15 Thread Andi Kleen
From: Andi Kleen Some projects need to prevent reordering of specific top level declarations with LTO, in particular declarations defining init calls. The only way to do that with LTO was to use -fno-toplevel-reorder, which stops reordering for all declarations and makes LTO partitioning less ef

Re: [GOOGLE] Fix LIPO COMDAT fixup and gcov-tool interactions

2014-09-15 Thread Xinliang David Li
Is it necessary to declare zero_counts array at all? Can a flag field be added to dyn_cgraph_node structure to indicate if it is fixed up? David On Fri, Sep 12, 2014 at 4:31 PM, Teresa Johnson wrote: > This patch addresses issues when running gcov-tool after performing > COMDAT fixup during dyn

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

2014-09-15 Thread Sebastian Huber
Ping. -- Sebastian Huber, embedded brains GmbH Address : Dornierstr. 4, D-82178 Puchheim, Germany Phone : +49 89 189 47 41-16 Fax : +49 89 189 47 41-09 E-Mail : sebastian.hu...@embedded-brains.de PGP : Public key available on request. Diese Nachricht ist keine geschäftliche Mitteilun

[PATCH i386 AVX512] [41/n] Extend extract insn patterns.

2014-09-15 Thread Kirill Yukhin
Hello, This patch extends extract insn patterns. It also fixes ICE on testsuite when F16C switched off. Also it fixes condition in old xtract pattern. Bootstrapped. AVX-512* tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ * config/i386/i386.c (ix86_ex

Re: [PR libfortran/62768] Handle filenames with embedded nulls

2014-09-15 Thread Janne Blomqvist
Ping. On Fri, Sep 5, 2014 at 3:54 AM, Janne Blomqvist wrote: > Hi, > > when opening a file with a name containing an ASCII null character, > the name is truncated at the first null since the IO API's take > null-terminated C strings. By storing such a C string in gfc_unit > instead of the origina

Re: Fix pr61848, linux kernel miscompile

2014-09-15 Thread Alan Modra
On Mon, Sep 15, 2014 at 08:36:58PM +, Joseph S. Myers wrote: > On Mon, 15 Sep 2014, Alan Modra wrote: > > > This patch cures the linux kernel boot failure when compiled using > > trunk gcc. (Andrew, apologies for hijacking your bugzilla, I started > > work on this before finding the bugzilla.

  1   2   >