Re: [PATCH] Support asan-instrumentation-with-call-threshold in GCC (second try)

> 1) instrument_mem_region_access/instrument_strlen should use a single >build_check_stmt call instead of two that they do now, I agree with instrument_mem_region_access but the strlen case is quite different because checks are inserted at different sides of instrumented insn. Adding yet mo

Re: [PATCH] Don't call init_caller_save if LRA enabled

2014-06-06 21:37 GMT+08:00 Vladimir Makarov : > On 2014-06-04, 12:31 PM, Kito Cheng wrote: >> >> LRA generate caller-save register store/restore during split register, >> not generate by caller-save.c:save_call_clobbered_regs, so initialize >> this module by init_caller_save is meaningless if LRA e

Move DECL_SECTION_NAME into symtab

Hi, this patch follows the change to move DECL_COMDAT_GROUP by moving DECL_SECTION_NAME into symtab nodes instead of keeping it in decl_with_vis. (I pla to proceed with other symbol table related fields). It follows exactly same path as the previous patch. Notable change is adding of node removal

Re: [Patch] Minor fixes for regtesting gfortran with -flto

On Mon, 5 May 2014, Dominique Dhumieres wrote: > With the following patch, gfortran can be regtested with -flto > with no failure, but pr54852 and pr60061. > > OK for trunk? > > Dominique > > 2014-05-05 Dominique d'Humieres > > * gfortran.dg/gfortran.dg/bind_c_array_params_2.f90: >

[fortran,patch] Binding label can be any initialisation expression

Well, only scalar character of the default kind, but still… This patch achieves this goal by following the obvious plan, which has not changed since I wrote it in PR 36275 in 2008 :) The custom matcher for binding label, in gfc_match_bind_c(), is removed and the generic matcher gfc_match_init_ex

Re: [PATCH] Trust TREE_ADDRESSABLE

> > ... In this particular translation unit you mean? > > Yes, in the translation unit being processed. > > > That would be worthless information for decls also reachable from elsewhere. > > It's the information: ADDR_EXPR of this DECL is taken somewhere in the IL, > it's no more or not less wo

Re: [PATCH] Fix PR middle-end/61141

On 06/08/14 12:06, John David Anglin wrote: On 4-Jun-14, at 3:38 AM, Jeff Law wrote: On 05/24/14 12:39, John David Anglin wrote: The insns are being deleted in "final" after the assembly output has being done. So, for example, pa_output_call is never called with a delayed branch sequence con

Re: [PATCH] Fix PR middle-end/61141

On 4-Jun-14, at 3:38 AM, Jeff Law wrote: On 05/24/14 12:39, John David Anglin wrote: The insns are being deleted in "final" after the assembly output has being done. So, for example, pa_output_call is never called with a delayed branch sequence containing a NOTE_INSN_DELETED. OK. That's go

Re: ipa-visibility TLC 2/n

Hi, this is the last part. It makes DECL_VIRTUAL to be copied when creating aliases. This is needed to make sanity check in gimple-fold happy (it checks that vtables are DECL_VIRTUAL). It also resets initializers of aliases to save memory. Bootstrapped/regtested rs6000-aix, comitted. Index: s

Re: ipa-visibility TLC 2/n

Hi, this is third part - it fixes handling of aliases in the anchor sections by recursing in place_block_symbol as discussed and also fixing one case where we lookup attribute instead of using symbol table * varasm.c (use_blocks_for_decl_p): Check symbol table instead of alias attr

Re: ipa-visibility TLC 2/n

Hi, this is second part of fix - it just makes ipa-visibility to work less hard when initializer already contains local aliases. I noticed this when debugging the original issue. Bootstrapped/regtested rs6000-aix, comitted. Honza * ipa-visibility.c: Include varasm.h (can_replac

Re: ipa-visibility TLC 2/n

> Honza, > > I finally was able to bootstrap GCC on AIX yesterday. The bootstrap > works, but the testsuite results are in extremely bad shape: 250 > failures in G++ testsuite and 470 failures in libstdc++ testsuite. David, this is first of the AIX fixes. It makes aliases to be output after thunk

[fortran,patch] Fix Cray pointers in modules

The attached one-line patch fixes PR45187 (https://gcc.gnu.org/bugzilla/show_bug.cgi?id=45187), where we would sometimes try to create the backend_decl of Cray-pointees twice. We should simply bail out early of the procedure, following what is done in the similar situation in gfc_finish_var_dec

[v3] Tighten some config/abi/pre/gnu.ver patterns

Hi, as discussed, tested x86_64-linux multilib, committed. Thanks, Paolo. /// 2014-06-08 Paolo Carlini * config/abi/pre/gnu.ver: Tighten some patterns; tidy. Index: config/abi/pre/gnu.ver === --- con

[PATCH, ARM] MI-thunk fix for TARGET_THUMB1_ONLY

Hi Richard, Ramana, Attached is a small fix for resolving a g++.old-deja/g++.jason/thunk2.C regression we found under a TARGET_THUMB1_ONLY multilib (-mthumb -march=armv6-m to be exact). Basically under those conditions, the thunk is in Thumb mode, so the subtraction should be 4 rather than 8. Ori

Re: [patch] fix tests for AVX512

On Tue, May 27, 2014 at 12:28 PM, Petr Murzin wrote: > Hi, > I've fixed tests for AVX512, so they could be compiled with -Werror > -Wall. Please have a look. > > > > 2014-05-19 Petr Murzin > > * gcc.target/i386/avx512f-vaddpd-2.c: Add static void for CALC, > void for TEST instead of static voi