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 enkovich@gmail.com: 2014-06-04 17:35 GMT+04:00 Richard Biener richard.guent...@gmail.com: On Wed, Jun 4, 2014 at 3:13 PM, Ilya Enkovich enkovich@gmail.com wrote: 2014-06-04 13:58 GMT+04:00 Richard Biener richard.guent...@gmail.com: On Wed,

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 richard.guent...@gmail.com: On Mon, Jun 2, 2014 at 5:15 PM, Ilya Enkovich enkovich@gmail.com wrote: Hi, In the most case we follow transparent alias chains wne assemble names. But in some cases it is not performed. For instrumented

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 enkovich@gmail.com: 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 ilya.enkov...@intel.com * config/i386/i386.c:

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 enkovich@gmail.com: 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 ilya.enkov...@intel.com

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 enkovich@gmail.com: 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 ilya.enkov...@intel.com *

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 enkovich@gmail.com: 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 ilya.enkov...@intel.com * config/i386/i386.c

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 enkovich@gmail.com: 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 ilya.enkov...@intel.com

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 enkovich@gmail.com: 2014-06-03 12:46 GMT+04:00 Richard Biener richard.guent...@gmail.com: On Mon, Jun 2, 2014 at 4:51 PM, Ilya Enkovich enkovich@gmail.com wrote: Hi, This patch adds support for normal builtin function calls (target ones

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 enkovich@gmail.com: On 03 Jun 17:27, Ilya Enkovich wrote: 2014-06-03 15:56 GMT+04:00 Richard Biener richard.guent...@gmail.com: On Tue, Jun 3, 2014 at 1:36 PM, Ilya Enkovich enkovich@gmail.com wrote: 2014-06-03 13:45 GMT+04:00 Richard

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 enkovich@gmail.com: On 03 Jun 11:54, Richard Biener wrote: On Tue, Jun 3, 2014 at 9:38 AM, Ilya Enkovich enkovich@gmail.com wrote: Hi, This patch allows BUILT_IN_CHKP_BNDRET as a consumer of a result of BUILT_IN_STACK_SAVE call.

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 enkovich@gmail.com: 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 enkovich@gmail.com: On 06 Jun 11:59, Ilya Enkovich wrote: 2014-06-03 13:07 GMT+04:00 Richard Biener richard.guent...@gmail.com: On Mon, Jun 2, 2014 at 5:56 PM, Ilya Enkovich enkovich@gmail.com wrote: + + /* If indirect call has

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 seg...@kernel.crashing.org 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

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. OK, so if

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 ilya.enkov...@intel.com * cgraph.h (cgraph_thunk_info): Add add_pointer_bounds_args field. (cgraph_node): Add instrumented_version, orig_decl and

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 enkovich@gmail.com wrote: Hi, This patch preserves CALL_WITH_BOUNDS flag for calls during PRE. Ok. Richard. Merging

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 a...@firstfloor.org wrote: From: Andi Kleen a...@linux.intel.com 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

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 a...@firstfloor.org wrote: From: Andi Kleen a...@linux.intel.com 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

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 al...@redhat.com 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

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 Thumb1 or

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 l...@redhat.com: 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. It also

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; { } }

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 david.sherw...@arm.com 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

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

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 dave.ang...@bell.net wrote: On 8-Sep-14, at 5:21 AM, Richard Biener wrote: On Sun, Sep 7, 2014 at 7:36 PM, John David Anglin dave.ang...@bell.net wrote: The attached patch fixes bootstrap on hpux which doesn't have the atoll function.

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 a...@codesourcery.com * gcc/config/arm/arm.c (arm_option_override):

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 seg...@kernel.crashing.org 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

[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 jwak...@redhat.com Date: Mon Sep 15 11:13:52 2014 +0100 * include/bits/regex.h (basic_regex): Rename _Ch_typeraits template parameters

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 bits/c++14_warning.h #else *** Jonathan: filesystem uses: #if __cplusplus 201103L # include bits/c++0x_warning.h #else and should probably

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 rguent...@suse.de * gimple-fold.c: Include tree-eh.c. (fold_stmt_1): Handle GIMPLE_CONDs for

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 than two

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

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 hardware and

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?)? Do we

[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 wdijk...@arm.com * gcc/config/aarch64/aarch64.c (aarch64_register_move_cost): Add register move costs for 128-bit types. --- gcc/config/aarch64/aarch64.c |

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 ilya.enkov...@intel.com * 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

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) +

[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 ilya.to...@intel.com * config/i386/driver-i386.c (host_detect_local_cpu):

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

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 tocarip.in...@gmail.com 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

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 ilya.enkov...@intel.com * tree-ssa-dce.c: Include tree-chkp.h. (propagate_necessity): For free call fed by alloc

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

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 ilya.enkov...@intel.com * ipa-split.c: Include tree-chkp.h. (find_retbnd): New. (consider_split): Do not split retbnd and retval producers. (split_function): Propagate Pointer Bounds

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 ilya.enkov...@intel.com * tree-ssa-ccp.c: Include tree-chkp.h. (insert_clobber_before_stack_restore): Handle BUILT_IN_CHKP_BNDRET calls. diff --git

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 a...@firstfloor.org wrote: From: Andi Kleen a...@linux.intel.com Test fentry and no_instrument_function overriding. No test cases for the LTO test for now, as the LTO harness

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 causes

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 ja...@redhat.com 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,

[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

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 l...@redhat.com: On 09/15/14 03:51, Ilya Enkovich wrote: 2014-08-15 Ilya Enkovich ilya.enkov...@intel.com * ipa-split.c: Include tree-chkp.h. (find_retbnd): New. (consider_split): Do not split retbnd and retval

[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 m...@gcc.gnu.org * 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 ja...@redhat.com 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.

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

2014-09-15 Thread Andi Kleen
Andi Kleen a...@firstfloor.org writes: Ping! From: Andi Kleen a...@linux.intel.com 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

[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 bur...@net-b.de: 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

Re: [PATCH] gcc parallel make check

2014-09-15 Thread Mike Stump
On Sep 15, 2014, at 9:05 AM, Jakub Jelinek ja...@redhat.com 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

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 ja...@redhat.com 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 in

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

[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

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 Burnusbur...@net-b.de: 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

[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

[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):

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 bur...@net-b.de: On 15.09.2014 19:18, Alessandro

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

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 sing...@google.com 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

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 sufficient

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 hongjiu...@intel.com

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

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 ilya.enkov...@intel.com * tree-ssa-ccp.c: Include tree-chkp.h. (insert_clobber_before_stack_restore): Handle

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 enkovich@gmail.com: 2014-06-04 17:35 GMT+04:00 Richard Biener richard.guent...@gmail.com: On Wed, Jun 4, 2014 at 3:13 PM, Ilya Enkovich enkovich@gmail.com wrote: 2014-06-04 13:58 GMT+04:00 Richard

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

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, it

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

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

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 excuse me, I am not quite

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 mikest...@comcast.net wrote: On Sep 15, 2014, at 9:13 AM, Jakub Jelinek ja...@redhat.com 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

[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.

[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

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 a...@linux.intel.com 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

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

2014-09-15 Thread Andi Kleen
From: Andi Kleen a...@linux.intel.com 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

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 tejohn...@google.com wrote: This patch addresses issues when running gcov-tool after performing

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

[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

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 blomqvist.ja...@gmail.com 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