Re: [committed] Remove condition from PA indirect_jump

2015-07-05 Thread Richard Sandiford
John David Anglin dave.ang...@bell.net writes: The attached change removes the C condition from the PA indirect jump since it depended on operands[] and this isn't generally allowed in named patterns. Tested on hppa-unknown-linux-gnu and hppa2.0w-hp-hpux11.11. Committed to trunk and active

Re: [patch] [fixincludes] Ignore .DS_Store junk files when running make check

2015-07-05 Thread Andreas Schwab
Eric Gallager eg...@gwmail.gwu.edu writes: I've attached trivial, 1-line patch to fixincludes_check.tpl; it allows 'make check' to succeed on OS X, by ignoring the files that Finder creates to keep track of the status of directories. -Eric Gallager fixincludes/check.tpl | 1 + 1 file

[committed] Use target-insns.def for trap

2015-07-05 Thread Richard Sandiford
Bootstrapped regression-tested on x86_64-linux-gnu and aarch64-linux-gnu. Also tested via config-list.mk. Committed as preapproved. Thanks, Richard gcc/ * target-insns.def (trap): New targetm instruction pattern. * builtins.c (expand_builtin_trap): Use it instead of

[committed] Use target-insns.def for prefetch

2015-07-05 Thread Richard Sandiford
Bootstrapped regression-tested on x86_64-linux-gnu and aarch64-linux-gnu. Also tested via config-list.mk. Committed as preapproved. Thanks, Richard gcc/ * target-insns.def (prefetch): New targetm instruction pattern. * tree-ssa-loop-prefetch.c: Include targeth.

[committed] Use target-insns.def for save/restore stack

2015-07-05 Thread Richard Sandiford
Bootstrapped regression-tested on x86_64-linux-gnu and aarch64-linux-gnu. Also tested via config-list.mk. Committed as preapproved. Thanks, Richard gcc/ * target-insns.def (restore_stack_block, restore_stack_function) (restore_stack_nonlocal, save_stack_block,

[committed] Use target-insns.def for insv, extv and extzv

2015-07-05 Thread Richard Sandiford
Bootstrapped regression-tested on x86_64-linux-gnu and aarch64-linux-gnu. Also tested via config-list.mk. Committed as preapproved. Thanks, Richard gcc/ * target-insns.def (extv, extzv, insv): New targetm instruction patterns. * optabs.c (get_extraction_insn): Use them

[committed] Use target-insns.def for untyped call/return

2015-07-05 Thread Richard Sandiford
Bootstrapped regression-tested on x86_64-linux-gnu and aarch64-linux-gnu. Also tested via config-list.mk. Committed as preapproved. Thanks, Richard gcc/ * target-insns.def (untyped_call, untyped_return): New targetm instruction patterns. * builtins.c

[committed] Use target-insns.def for forms of non-local goto

2015-07-05 Thread Richard Sandiford
Bootstrapped regression-tested on x86_64-linux-gnu and aarch64-linux-gnu. Also tested via config-list.mk. Committed as preapproved. Thanks, Richard gcc/ * target-insns.def (builtin_longjmp, builtin_setjmp_receiver) (builtin_setjmp_setup, exception_receiver, nonlocal_goto)

[committed] Use target-insns.def for stack_protect_{set,test}

2015-07-05 Thread Richard Sandiford
Bootstrapped regression-tested on x86_64-linux-gnu and aarch64-linux-gnu. Also tested via config-list.mk. Committed as preapproved. Thanks, Richard gcc/ * target-insns.def (stack_protect_set, stack_protect_test): New targetm instruction patterns. * cfgexpand.c

[committed] Use target-insns.def for doloop_{begin,end}

2015-07-05 Thread Richard Sandiford
Bootstrapped regression-tested on x86_64-linux-gnu and aarch64-linux-gnu. Also tested via config-list.mk. Committed as preapproved. Thanks, Richard gcc/ * target-insns.def (doloop_begin, doloop_end): New targetm instruction patterns. * loop-init.c: Include target.h.

[committed] Use target-insns.def for various stack patterns

2015-07-05 Thread Richard Sandiford
Bootstrapped regression-tested on x86_64-linux-gnu and aarch64-linux-gnu. Also tested via config-list.mk. Committed as preapproved. Thanks, Richard gcc/ * target-insns.def (allocate_stack, check_stack, probe_stack) (probe_stack_address, split_stack_prologue,

[committed] Use target-insns.def for clear_cache

2015-07-05 Thread Richard Sandiford
Bootstrapped regression-tested on x86_64-linux-gnu and aarch64-linux-gnu. Also tested via config-list.mk. Committed as preapproved. Thanks, Richard gcc/ * target-insns.def (clear_cache): New targetm instruction pattern. * builtins.c (expand_builtin___clear_cache): Use it

[committed] Remove gen_move_insn_uncast

2015-07-05 Thread Richard Sandiford
The previous patch removed the only users of gen_move_insn_uncast (which was added to make those functions work with rtx_insn *). Bootstrapped regression-tested on x86_64-linux-gnu and aarch64-linux-gnu. Also tested via config-list.mk. Committed as obvious. Thanks, Richard gcc/ *

fix segfault in verify_flow_info() with -dx option

2015-07-05 Thread Prathamesh Kulkarni
Hi, Passing -dx causes segmentation fault: Test case: void f(void) {} ./test.c: In function 'f': ../test.c:3:1: internal compiler error: Segmentation fault } ^ 0xab6baf crash_signal /home/prathamesh.kulkarni/gnu-toolchain/src/gcc.git/gcc/toplev.c:366 0x694b14 verify_flow_info()

flatten cfgloop.h

2015-07-05 Thread Prathamesh Kulkarni
Hi, The attached patches flatten cfgloop.h. patch-1.diff moves around prototypes and structures to respective header-files. patch-2.diff (mostly auto-generated) replicates cfgloop.h includes in c files. Bootstrapped and tested on x86_64-unknown-linux-gnu with all front-ends. Built on all targets

Re: [PATCH 1/2][ARM] PR/65956 AAPCS update for alignment attribute

2015-07-05 Thread Eric Botcazou
Technically this is incorrect since AGGREGATE_TYPE_P includes ARRAY_TYPE and ARRAY_TYPE doesn't have TYPE_FIELDS. I doubt we could reach that case though (unless there's a language that allows passing arrays by value). Ada passes small array types by the method specified by the

[fortran, patch] Remove libgfortranbegin.a

2015-07-05 Thread FX
Hi, 6 years ago, we removed the static helper library libgfortranbegin.a. Since gfortran 4.5, we don’t use it, but still provide it for backward compatibility. I’m thinking 6 years notice is enough, and we should remove it. A search for it on github and other code repositories does not

Re: [Ping, Patch, Fortran, PR58586, v5] ICE with derived type with allocatable component passed by value

2015-07-05 Thread Paul Richard Thomas
Dear Andre, I agree with Steve's recommendation that you comment out the line and open a PR for the problem. The patch looks fine to me and applied cleanly, apart from trailing CRs in the testcases. OK by me too. Cheers Paul PS I felt safe in setting a deadline for the submodule patch

Re: [PATCH][8/n] Remove GENERIC stmt combining from SCCVN

2015-07-05 Thread Eric Botcazou
The following moves some bitwise patterns from the match-and-simplify branch, extending them with proper conditional converts and removing the corresponding patterns from fold-const.c There is a pasto/thinko in the last pattern: /* Convert ~X ^ C to X ^ ~C. */ (simplify (bit_xor (convert?

Re: [fortran, patch] Remove libgfortranbegin.a

2015-07-05 Thread Steve Kargl
On Sun, Jul 05, 2015 at 03:37:25PM +0200, FX wrote: Hi, 6 years ago, we removed the static helper library libgfortranbegin.a. Since gfortran 4.5, we don???t use it, but still provide it for backward compatibility. I???m thinking 6 years notice is enough, and we should remove it. A

[PATCH, fortran] Fix numerous ICEs in IO statements with IOMSG

2015-07-05 Thread Steve Kargl
The attached patch fixes numerous ICE's where a program writed invalid code (see testcase). Regression tested on i386-*-freebsd. OK to commit? 2015-07-05 Steven G. Kargl ka...@gcc.gnu.org * io.c (check_char_variable): New function. (match_open_element, match_close_element,

[MOXIE] Hookize GO_IF_LEGITIMATE_ADDRESS

2015-07-05 Thread Anatoliy Sokolov
Hi. This patch removes obsolete GO_IF_LEGITIMATE_ADDRESS macros from the MOXIE back end in the GCC and introduces equivalent TARGET_ADDR_SPACE_LEGITIMATE_ADDRESS_P target hook. Regression tested on moxie-unknown-elf. OK for trunk? 2015-07-05 Anatoly Sokolov ae...@post.ru *

Re: [Ping, Patch, Fortran, PR58586, v5] ICE with derived type with allocatable component passed by value

2015-07-05 Thread Steve Kargl
On Sat, Jul 04, 2015 at 09:20:39PM +0200, Andre Vehreschild wrote: Thanks for looking at the code. The error you experience is known to me. The bug is present in gfortran and only exposed by this patch. Unfortunately is the pr58586 not addressing this specific error. It may be in the

Re: [Ping, Patch, Fortran, PR58586, v5] ICE with derived type with allocatable component passed by value

2015-07-05 Thread Steve Kargl
On Sun, Jul 05, 2015 at 07:48:13PM +0200, Paul Richard Thomas wrote: Dear Andre, I agree with Steve's recommendation that you comment out the line and open a PR for the problem. The patch looks fine to me and applied cleanly, apart from trailing CRs in the testcases. OK by me too.

[PATCH] PR target/35514: Gcc shoud generate symbol type for undefined symbol

2015-07-05 Thread H.J. Lu
Update default_elf_asm_output_external to also output symbol type to help ELF linker to properly issue diagnostic message. We don't output symbol type for reference to external TLS symbol since assembler will generate TLS symbol type based on TLS relocation and Solaris assembler only supports the

Re: [PATCH][11/n] Remove GENERIC stmt combining from SCCVN

2015-07-05 Thread H.J. Lu
On Thu, Jul 2, 2015 at 7:05 AM, Richard Biener rguent...@suse.de wrote: This moves floating-point related comparison foldings from fold_comparison to match.pd. I noticed we call fold_comparison only for a subset of tcc_comparison - the newly introduced simple_comparison operator list and

[wwwdocs] debian.org now uses https

2015-07-05 Thread Gerald Pfeifer
...so convert to links in our manuals. Applied. Gerald 2015-07-05 Gerald Pfeifer ger...@pfeifer.com * doc/fragments.texi (Target Fragment): Convert debian.org link to use https. * doc/install.texi (Configuration): Ditto. Index: doc/fragments.texi

Re: [PATCH] PR target/35514: Gcc shoud generate symbol type for undefined symbol

2015-07-05 Thread H.J. Lu
On Sun, Jul 5, 2015 at 11:14 AM, H.J. Lu hjl.to...@gmail.com wrote: Update default_elf_asm_output_external to also output symbol type to help ELF linker to properly issue diagnostic message. We don't output symbol type for reference to external TLS symbol since assembler will generate TLS

Get rid of move_iterator in debug checks

2015-07-05 Thread François Dumont
Hi I would like to restore foreign iterator check even when move_iterator are being use. To do so I leverage on __miter_base to get rid of move_iterator layer before running the check. I had to expose __miter_base fallback implementation in cpp_type_traits.h to safely use it in debug mode.

[PATCH] PR target/53383: Allow -mincoming-stack-boundary=3 with -mno-sse

2015-07-05 Thread H.J. Lu
Similar to -mpreferred-stack-boundary=3, -mincoming-stack-boundary=3 is allowed with -mno-sse in 64-bit mode. OK for trunk? H.J. gcc/ PR target/53383 * config/i386/i386.c (ix86_option_override_internal): Allow -mincoming-stack-boundary=X if -mpreferred-stack-boundary=N

[nios2, committed] allow constant offset with %gprel

2015-07-05 Thread Sandra Loosemore
I've checked in the attached patch to fix an oversight in the handling of GP-relative addressing modes in the nios2 back end. It allows GCC to generate addresses of the form %gprel(sym+offset)(gp) instead of just %gprel(sym)(gp). Formerly GCC required a scratch register and extra instruction

Re: [patch] [fixincludes] Ignore .DS_Store junk files when running make check

2015-07-05 Thread Eric Gallager
I was just matching the code that was already used there... should the lines to ignore the CVS and .svn folders be re-written into the style you propose, too? On 7/5/15, Andreas Schwab sch...@linux-m68k.org wrote: Eric Gallager eg...@gwmail.gwu.edu writes: I've attached trivial, 1-line patch

Re: [PATCH 1/2] Allow REG_EQUAL for ZERO_EXTRACT

2015-07-05 Thread Kugan
On 30/06/15 17:16, Maxim Kuvyrkov wrote: On Jun 30, 2015, at 6:54 AM, Kugan kugan.vivekanandara...@linaro.org wrote: On 29/06/15 21:56, Maxim Kuvyrkov wrote: On Jun 28, 2015, at 2:28 PM, Kugan kugan.vivekanandara...@linaro.org wrote: This patch allows setting REG_EQUAL for ZERO_EXTRACT

Re: [PATCH] New configure option to default enable Smart Stack Protection

2015-07-05 Thread Magnus Granberg
fredag 03 juli 2015 01.01.51 skrev Magnus Granberg: Hi Working on a patch that enable Smart Stack Protection as default. I still miss docs and testcase for the patch. I need you ides and help. /Magnus G. --- New patch with doc and testcase added. /Magnus G. Changlogs /gcc 2015-07-05