RE: [PATCH, ARM] Backport fix for PR59593 (minipool of small values on big endian targets)

2015-02-10 Thread Thomas Preud'homme
Ping? > -Original Message- > From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- > ow...@gcc.gnu.org] On Behalf Of Thomas Preud'homme > Sent: Tuesday, January 20, 2015 1:06 PM > To: gcc-patches@gcc.gnu.org; Richard Earnshaw; Ramana Radhakrishnan; > Marcus Shawcroft > Subject: [PATCH,

Re: [PATCH][libstdc++][Testsuite] isctype test fails for newlib.

2015-02-10 Thread Matthew Wahab
On 09/02/15 23:17, Hans-Peter Nilsson wrote: On Mon, 9 Feb 2015, Matthew Wahab wrote: Attached a patch to replace the test for macro __NEWLIB__ with a test for macro NEWLINE_IN_CLASS_BLANK, defined by { dg-additional-options "-DNEWLINE_IN_CLASS_BLANK" { target newlib } } Tested with check-tar

Re: [PATCH][AArch64] Fix illegal assembly 'eon v1, v2, v3'

2015-02-10 Thread James Greenhalgh
On Wed, Jan 28, 2015 at 02:04:04PM +, James Greenhalgh wrote: > On Wed, Jan 28, 2015 at 12:32:45PM +, Alan Lawrence wrote: > > Ok for stage 4? > > This is a regression from 4.9, so once we iron out some nits, it should > be. > > > gcc/ChangeLog: > > > > * config/aarch64/aarch64.md (*

Re: [PATCH][ARM] PR target/64600 Fix another ICE with -mtune=xscale: properly sign-extend mask during constant splitting

2015-02-10 Thread Kyrill Tkachov
Ping. https://gcc.gnu.org/ml/gcc-patches/2015-02/msg00141.html Thanks, Kyrill On 03/02/15 15:18, Kyrill Tkachov wrote: Hi all, The ICE in this PR occurs when -mtune=xscale triggers a particular path through arm_gen_constant during expand that creates a 0xf00f mask but for a 64-bit HOST_WI

[PATCH] Fix PR64909

2015-02-10 Thread Richard Biener
The vectorizer cost model has a serious issue in not dealing well with targets using scalar stmt cost != 1. This is because it passes scalar iteration _cost_ to routines scaling that cost with the targets scalar stmt cost again. This is for example visible on x86_64 for all AMD archs which use h

[PATCH] Remove unused return value of streamer_read_tree_bitfields

2015-02-10 Thread Richard Biener
LTO bootstrapped on x86_64-unknown-linux-gnu, applied. Richard. 2015-02-10 Richard Biener * tree-streamer.h (streamer_read_tree_bitfields): Adjust. * tree-streamer-in.c (streamer_read_tree_bitfields): Do not return the bitpack. Index: gcc/tree-streamer.h

Re: [SH] Allow reg+disp address modes for atomics

2015-02-10 Thread Kaz Kojima
Oleg Endo wrote: > The attached patch fixes the lost mem aliasing info for atomic ops on SH > and allows the utilization of reg+disp address modes for atomic ops. > Actually it was supposed to be a pretty straight forward patch that just > replaces the open coded 'mem:QIHISI (match_operand:SI > "a

Re: [PATCH PR64705]Don't aggressively expand induction variable's base

2015-02-10 Thread Richard Biener
On Mon, Feb 9, 2015 at 11:33 AM, Bin Cheng wrote: > The second time I missed patch in one day, I hate myself. > Here it is. I think the patch is reasonable but I would have used a default = NULL arg for 'stop' to make the patch smaller. You don't constrain 'stop' to being an SSA name - any parti

Re: [PATCH][RFC][OpenMP] Forbid target* pragmas in target regions

2015-02-10 Thread Martin Jambor
Hi, On Mon, Jan 12, 2015 at 12:22:44AM +0300, Ilya Verbin wrote: > Hi! > > Currently if a target* pragma appears within a target region, GCC successfully > compiles such code (with a warning). But the binary fails at run-time, since > it > tries to call GOMP_target* functions on target. > > Th

Re: RFA: RL78: Fix gcc testsuite failures

2015-02-10 Thread Nicholas Clifton
Hi Jan-Benedict, Seems you accidentally committed quite some more code you're currently working on in that very commit, which is now breaking Doh! Yes, sorry about that. I have now reverted the DIV attribute part of the patch. (I am hoping that this work will be ready for contributing soo

Re: [PATCH PR64705]Don't aggressively expand induction variable's base

2015-02-10 Thread Bin.Cheng
On Tue, Feb 10, 2015 at 6:02 PM, Richard Biener wrote: > On Mon, Feb 9, 2015 at 11:33 AM, Bin Cheng wrote: >> The second time I missed patch in one day, I hate myself. >> Here it is. > > I think the patch is reasonable but I would have used a default = NULL > arg for 'stop' to make the patch smal

Re: [PATCH][RFC][OpenMP] Forbid target* pragmas in target regions

2015-02-10 Thread Jakub Jelinek
On Tue, Feb 10, 2015 at 11:16:22AM +0100, Martin Jambor wrote: > On Mon, Jan 12, 2015 at 12:22:44AM +0300, Ilya Verbin wrote: > > Currently if a target* pragma appears within a target region, GCC > > successfully > > compiles such code (with a warning). But the binary fails at run-time, > > sinc

Re: Another ptx offloading patch

2015-02-10 Thread Richard Biener
On Wed, Feb 4, 2015 at 11:38 AM, Jakub Jelinek wrote: > On Fri, Jan 16, 2015 at 11:45:40AM +0100, Richard Biener wrote: >> On Thu, Jan 15, 2015 at 7:35 PM, Jeff Law wrote: >> > On 11/20/14 05:33, Bernd Schmidt wrote: >> >> >> >> Now that I've managed to put together and test all the submitted Ope

Re: [PATCH PR64705]Don't aggressively expand induction variable's base

2015-02-10 Thread Bin.Cheng
On Tue, Feb 10, 2015 at 6:18 PM, Bin.Cheng wrote: > On Tue, Feb 10, 2015 at 6:02 PM, Richard Biener > wrote: >> On Mon, Feb 9, 2015 at 11:33 AM, Bin Cheng wrote: >>> The second time I missed patch in one day, I hate myself. >>> Here it is. >> >> I think the patch is reasonable but I would have u

Re: [PATCH][RFC][OpenMP] Forbid target* pragmas in target regions

2015-02-10 Thread Thomas Schwinge
Hi! On Tue, 10 Feb 2015 11:20:24 +0100, Jakub Jelinek wrote: > On Tue, Feb 10, 2015 at 11:16:22AM +0100, Martin Jambor wrote: > > On Mon, Jan 12, 2015 at 12:22:44AM +0300, Ilya Verbin wrote: > > > Currently if a target* pragma appears within a target region, GCC > > > successfully > > > compiles

RE: [PATCH IRA] update_equiv_regs fails to set EQUIV reg-note for pseudo with more than one definition

2015-02-10 Thread Ajit Kumar Agarwal
-Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org] On Behalf Of Jeff Law Sent: Tuesday, February 10, 2015 5:03 AM To: Bin.Cheng Cc: Alex Velenko; Felix Yang; Yangfei (Felix); Marcus Shawcroft; GCC Patches; vmaka...@redhat.com Subject: Re: [PATC

Re: [debug-early] C++ clones and limbo DIEs

2015-02-10 Thread Richard Biener
On Fri, Feb 6, 2015 at 5:42 PM, Aldy Hernandez wrote: > >>> +&& DECL_CONTEXT (snode->decl) >>> +&& TREE_CODE (DECL_CONTEXT (snode->decl)) != FUNCTION_DECL) >> >> >> I think this should be !decl_function_context (snode->decl), in case >> there's a class or BLOCK between the symbol and its e

[PATCH] Fix PR ipa/64813

2015-02-10 Thread Martin Liška
Hello. Following patch is fix for PR ipa/64813. The patch was tested on a darwin target by Dominique. Ready for trunk? Thanks, Martin >From ce1c7031e2616d840ce9c1bc45587d64134f Mon Sep 17 00:00:00 2001 From: mliska Date: Fri, 30 Jan 2015 15:12:59 +0100 Subject: [PATCH] Handle noreturn funct

[wwwdocs] Add Links section to porting to

2015-02-10 Thread Marek Polacek
We've performed a mass rebuild for Fedora, so I'm updating the porting to document with the following. Index: porting_to.html === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-5/porting_to.html,v retrieving revision 1.2 diff -u -r1.2 porting_

Re: [PATCH] Fix PR64764 && Fix scan-tree-dump for scop-19.c

2015-02-10 Thread Tom de Vries
On 09-02-15 20:01, Dominique d'Humières wrote: Le 9 févr. 2015 à 19:10, Tom de Vries mailto:tom_devr...@mentor.com>> a écrit : On 09-02-15 18:23, Dominique Dhumieres wrote: Tom, After these changes I have the following regressions on x86_64-apple-darwin1[04]*: FAIL: gcc.dg/uninit-19.c (tes

Re: [RFC testsuite] Fix PR64850, tweak acc_on_device* tests

2015-02-10 Thread Thomas Schwinge
Hi! On Wed, 4 Feb 2015 10:40:40 +0100, I wrote: > On Wed, 04 Feb 2015 08:41:28 +0900 (JST), Kaz Kojima > wrote: > > Several goacc/acc_on_device tests fail for a few targets: > > > > hppa2.0w-hp-hpux11.11 (PR testsuite/64850) > > https://gcc.gnu.org/ml/gcc-testresults/2015-01/msg02659.html > >

Re: Another ptx offloading patch

2015-02-10 Thread Bernd Schmidt
On 02/10/2015 11:34 AM, Richard Biener wrote: Bernds patch is ok with the unrelated gcc.c hunk removed. Thanks. I'm assuming this is for the next stage 1? Bernd

Re: Another ptx offloading patch

2015-02-10 Thread Jakub Jelinek
On Tue, Feb 10, 2015 at 01:06:15PM +0100, Bernd Schmidt wrote: > On 02/10/2015 11:34 AM, Richard Biener wrote: > > >Bernds patch is ok with the unrelated gcc.c hunk removed. > > Thanks. I'm assuming this is for the next stage 1? No, for current trunk, we don't want to ship with offloading to nvp

Re: [PATCH] Fix PR ipa/64813

2015-02-10 Thread Richard Biener
On Tue, Feb 10, 2015 at 12:07 PM, Martin Liška wrote: > Hello. > > Following patch is fix for PR ipa/64813. The patch was tested on a darwin > target > by Dominique. > > Ready for trunk? - if (!(gimple_call_flags (call) & ECF_NORETURN)) + if (!alias_is_noreturn) that was technically un

[PATCH] Fix PR64995

2015-02-10 Thread Richard Biener
The following fixes PR64955 where we forgot to valueize VOPs before using them as value-number. Bootstrapped on x86_64-unknown-linux-gnu, testing in progress. Richard. 2015-10-10 Richard Biener PR tree-optimization/64995 * tree-ssa-sccvn.c (set_ssa_val_to): Assert that the

Re: [Haifa Scheduler] Fix latent bug in macro-fusion/instruction grouping

2015-02-10 Thread Maxim Kuvyrkov
On Feb 10, 2015, at 7:16 AM, Jeff Law wrote: > On 02/06/15 05:24, James Greenhalgh wrote: >> >> --- >> 2015-02-06 James Greenhalgh >> >> * haifa-sched.c (recompute_todo_spec): After applying a >> replacement and cancelling a dependency, also clear the >> SCHED_GROUP_P flag. >

[PATCH] cap frequency in make_forwarder_block PR64326

2015-02-10 Thread tbsaunde+gcc
From: tbsaunde In the testcase the block with problematic frequency has to incoming edges. Each edge has probability 1. The first edge's source bb has frequency 873/1, and the second has a frequency of 9409/1. So at least one of those is slightly high probably from some sort of roundin

Re: [PATCH IRA] update_equiv_regs fails to set EQUIV reg-note for pseudo with more than one definition

2015-02-10 Thread Jeff Law
On 02/10/15 03:51, Ajit Kumar Agarwal wrote: R110 can have the value -1 (BB6, BB7, BB8) or 0 (BB5). Thus there is no single value across the entire function that one can validly use for r110. Does the value of R110 should not change across all the callee path from the given caller functions.

Re: [PATCH] Fix PR ipa/64813

2015-02-10 Thread Martin Liška
On 02/10/2015 01:50 PM, Richard Biener wrote: On Tue, Feb 10, 2015 at 12:07 PM, Martin Liška wrote: Hello. Following patch is fix for PR ipa/64813. The patch was tested on a darwin target by Dominique. Ready for trunk? - if (!(gimple_call_flags (call) & ECF_NORETURN)) + if (!alias

Re: [PATCH] Fix PR ipa/64813

2015-02-10 Thread Jakub Jelinek
On Tue, Feb 10, 2015 at 04:56:40PM +0100, Martin Liška wrote: > On 02/10/2015 01:50 PM, Richard Biener wrote: > >On Tue, Feb 10, 2015 at 12:07 PM, Martin Liška wrote: > >>Hello. > >> > >>Following patch is fix for PR ipa/64813. The patch was tested on a darwin > >>target > >>by Dominique. > >> > >

[Patch Testsuite] XFAIL gfortran.dg/pr45636.f90 on AArch64

2015-02-10 Thread James Greenhalgh
Hi, As is already done for mips and hppa, we should XFAIL this test on AArch64 as we don't currently use the store_by_pieces infrastructure. We may in future want to tweak this, but for GCC 5.0 the safe thing to do is just to XFAIL the test. Tested with aarch64-none-elf. OK? Thanks, James --

New Esperanto PO file for 'cpplib' (version 5.1-b20150208)

2015-02-10 Thread Translation Project Robot
Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'cpplib' has been submitted by the Esperanto team of translators. The file is available at: http://translationproject.org/latest/cpplib/eo.po (This file, 'cpplib-5.1-b201502

Contents of PO file 'cpplib-5.1-b20150208.eo.po'

2015-02-10 Thread Translation Project Robot
cpplib-5.1-b20150208.eo.po.gz Description: Binary data The Translation Project robot, in the name of your translation coordinator.

Mark one forgotten optimization flag

2015-02-10 Thread Jan Hubicka
Hi, this patch adds optimization attribute to fstack-reuse that is obviously per-function. Bootstrapped/regtested x86_64-linux, commited as obvious. Honza Index: common.opt === --- common.opt (revision 220586) +++ common.opt (work

Re: [PATCH] PR64959: SFINAE in UDLs

2015-02-10 Thread Jason Merrill
On 02/09/2015 08:27 AM, Andrea Azzarone wrote: Please note that this is my first gcc patch Thanks, looks good! A couple of nits: * gcc/cp/parser.c: Make sure lookup_literal_operator returns all the possible candidates. Also improve the diagnostic messages. A ChangeLog entry should be p

Re: [Patch Testsuite] XFAIL gfortran.dg/pr45636.f90 on AArch64

2015-02-10 Thread Mike Stump
On Feb 10, 2015, at 8:06 AM, James Greenhalgh wrote: > As is already done for mips and hppa, we should XFAIL this test on > AArch64 as we don't currently use the store_by_pieces infrastructure. For this patch, either this is obvious or a target person should weigh in. I’d only step in if the ta

Re: [debug-early] C++ clones and limbo DIEs

2015-02-10 Thread Jason Merrill
On 02/10/2015 05:52 AM, Richard Biener wrote: I also wonder why you restrict it to functions with a GIMPLE body. Likewise for struct foo { int i; }; I guess these should depend on -feliminate-unused-debug-symbols and -feliminate-unused-debug-types. Jason

[PATCH, alpha]: Fix reload_out_aligned insn pattern

2015-02-10 Thread Uros Bizjak
Hello! gfortran.dg/char_length_15.f90 FAILs on non-BWX alpha due to invalid propagation of memory address into reload_out_aligned insn pattern. We start with: *** char_length_15.f90.228r.reload: (insn 577 456 460 14 (parallel [ (set (mem/c:QI (plus:DI (reg:DI 6 $6 [358])

[wwwdocs] Some stuff for porting to

2015-02-10 Thread Marek Polacek
The following adds a few new notes. Ok? --- porting_to.html 10 Feb 2015 11:12:20 - 1.3 +++ porting_to.html 10 Feb 2015 17:58:52 - @@ -251,6 +251,99 @@ ^ +__STDC_VERSION__ macro + +In the C11 mode, the __STDC_VERSION__ standard macro, +introduced in

Re: [PATCH] Fix PR64764 && Fix scan-tree-dump for scop-19.c

2015-02-10 Thread Jack Howarth
Tom, At -m32 on x86_64-apple-darwin14, the failing test case executes... % /sw/src/fink.build/gcc50-5.0.0-1000/darwin_objdir/gcc/xgcc -B/sw/src/fink.build/gcc50-5.0.0-1000/darwin_objdir/gcc/ /sw/src/fink.build/gcc50-5.0.0-1000/gcc-5-20150209/gcc/testsuite/gcc.dg/uninit-19.c -fno-diagnostics

Re: [PATCH] PR rtl-optimization/32219: optimizer causes wrong code in pic/hidden/weak symbol checking

2015-02-10 Thread Jack Howarth
Mike, Actually there aren't really any darwin bits in the current patch... https://gcc.gnu.org/ml/gcc-patches/2015-02/msg00468.html short of the dg-skip-if adjustments for the unsupported targets. Jack ps I wonder if this final patch needs be reposted to gcc-patches as...

Re: [PATCH] Implement libffi for AARCH64:ILP32

2015-02-10 Thread Richard Henderson
On 02/09/2015 12:40 AM, Andrew Pinski wrote: > #ifndef LIBFFI_ASM > +#ifdef __ILP32__ > +typedef unsigned long long ffi_arg; > +typedef signed long long ffi_sarg; > +#else You need to set FFI_SIZEOF_ARG too. Otherwise it looks ok. r~

Re: [PATCH] Implement libffi for AARCH64:ILP32

2015-02-10 Thread Richard Henderson
On 02/09/2015 12:51 AM, Andrew Haley wrote: > Would it make more sense to use int64_t rather than long long? Probably, but we should do that all over libffi, rather than just this one hunk. r~

Re: PR 63566 part 5

2015-02-10 Thread H.J. Lu
On Mon, Feb 9, 2015 at 12:50 PM, Jan Hubicka wrote: > Hi, > this patch finally disables the wroarkound in ipa-icf for local functions. > > Bootstrapped/regtested x86_64-linux, comitted. > > Honza > > 2015-02-08 Jan Hubicka > > PR ipa/63566 > * ipa-icf.c (set_local): New function

Re: [PATCH] Implement libffi for AARCH64:ILP32

2015-02-10 Thread Andrew Pinski
On Tue, Feb 10, 2015 at 10:35 AM, Richard Henderson wrote: > On 02/09/2015 12:40 AM, Andrew Pinski wrote: >> #ifndef LIBFFI_ASM >> +#ifdef __ILP32__ >> +typedef unsigned long long ffi_arg; >> +typedef signed long long ffi_sarg; >> +#else > > You need to set FFI_SIZEOF_ARG too. > > Otherwise it lo

[C++ PATCH] Don't ICE on internal calls in check_noexcept_r (PR sanitizer/64984)

2015-02-10 Thread Jakub Jelinek
Hi! Internal calls never throw and as shown on the testcase, sometimes they can make it all the way to check_noexcept_r which was ICEing on those. Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? 2015-02-10 Jakub Jelinek PR sanitizer/64984 * except.c (chec

[PATCH] Fix -fsanitize=vptr sanopt lowering (PR sanitizer/65004)

2015-02-10 Thread Jakub Jelinek
Hi! As ubsan_expand_vptr_ifn points *gsip to the first stmt in the next bb, we want to always return true as no_next, we never want to gsi_next on it before processing it further, otherwise we could skip some important statement (e.g. another UBSAN_* call). Fixed thusly, bootstrapped/regtested on

Re: [PATCH] Fix PR ipa/64813

2015-02-10 Thread Jan Hubicka
> Hello. > > Following patch is fix for PR ipa/64813. The patch was tested on a darwin > target > by Dominique. > > Ready for trunk? OK, Honza > Thanks, > Martin > >From ce1c7031e2616d840ce9c1bc45587d64134f Mon Sep 17 00:00:00 2001 > From: mliska > Date: Fri, 30 Jan 2015 15:12:59 +0100 > S

Re: Fix PR 63566 part 4

2015-02-10 Thread H.J. Lu
On Sun, Feb 8, 2015 at 1:10 PM, Jan Hubicka wrote: > Hi, > this patch finally enables local functions for functions with aliases and > thunks. > > Honza > > PR ipa/63566 > * ipa-visibility.c (cgraph_node::non_local_p): Accept aliases. > (cgraph_node::local_p): Remove thunk

Re: [SH] Allow reg+disp address modes for atomics

2015-02-10 Thread Oleg Endo
On Tue, 2015-02-10 at 18:35 +0900, Kaz Kojima wrote: > Oleg Endo wrote: > > The attached patch fixes the lost mem aliasing info for atomic ops on SH > > and allows the utilization of reg+disp address modes for atomic ops. > > Actually it was supposed to be a pretty straight forward patch that just

[RFA][PATCH][PR rtl-optimization/47477] Type narrowing in match.pd

2015-02-10 Thread Jeff Law
This PR was originally minor issue where we regressed on this kind of sequence: typedef struct toto_s *toto_t; toto_t add (toto_t a, toto_t b) { int64_t tmp = (int64_t)(intptr_t)a + ((int64_t)(intptr_t)b&~1L); return (toto_t)(intptr_t) tmp; } There was talk of trying to peephole this in

Re: [PATCH] PR rtl-optimization/32219: optimizer causes wrong code in pic/hidden/weak symbol checking

2015-02-10 Thread Richard Henderson
On 02/07/2015 08:45 AM, H.J. Lu wrote: > Here is an updated patch. This is an annoying comment to differentiate 3 different versions, FYI. > @@ -6826,11 +6817,18 @@ default_binds_local_p_1 (const_tree exp, int shlib) >&& (TREE_STATIC (exp) || DECL_EXTERNAL (exp))) > { >varpoo

Re: [PATCH] Implement libffi for AARCH64:ILP32

2015-02-10 Thread Richard Henderson
On 02/10/2015 11:02 AM, Andrew Pinski wrote: > +#define FFI_SIZEOF_JAVA_RAW 4 Since aarch64 doesn't implement the raw java api, this is irrelevant. r~

Re: [PATCH] PR rtl-optimization/32219: optimizer causes wrong code in pic/hidden/weak symbol checking

2015-02-10 Thread H.J. Lu
On Tue, Feb 10, 2015 at 1:19 PM, Richard Henderson wrote: >> @@ -7445,9 +7465,10 @@ default_elf_asm_output_external (FILE *file >> ATTRIBUTE_UNUSED, >> { >>/* We output the name if and only if TREE_SYMBOL_REFERENCED is >> set in order to avoid putting out names that are never really >>

Re: [PATCH] Fix -fsanitize=vptr sanopt lowering (PR sanitizer/65004)

2015-02-10 Thread Richard Biener
On February 10, 2015 9:08:33 PM CET, Jakub Jelinek wrote: >Hi! > >As ubsan_expand_vptr_ifn points *gsip to the first stmt in the next bb, >we want to always return true as no_next, we never want to gsi_next on >it >before processing it further, otherwise we could skip some important >statement (e.

Re: patch to fix rtl documentation for new floating point comparisons

2015-02-10 Thread Joseph Myers
On Mon, 9 Feb 2015, Kenneth Zadeck wrote: > > I don't think it's useful to have the trapping semantics unspecified for a > > comparison operation like that. So the question is what's most useful for > > LTGT and LTGT_EXPR to do (presumably they should do the same thing). Lots > > of existing cod

Re: patch to fix rtl documentation for new floating point comparisons

2015-02-10 Thread Eric Botcazou
> But maybe more code treats LTGT as a signaling operation than otherwise, in > accordance with the original intent, and so that's the most appropriate way > to document it (with !UNEQ being the representation of the corresponding > quiet operation). SPARC supports all codes and treats everything

Re: [wwwdocs] Some stuff for porting to

2015-02-10 Thread Joseph Myers
On Tue, 10 Feb 2015, Marek Polacek wrote: > +Several new warnings have been added to the C front end. One of the new > +warnings is that GCC now warns about non-standard predefined macros with the > +-Wpedantic option. For instance: > + > + > + void > + foo (void) > + { > +const char *s =

Re: [Patch, fortran] PR64952 - Missing temporary in assignment from elemental function

2015-02-10 Thread Paul Richard Thomas
Dear Mikael, dear all, Thank you for the previous review. I believe that the attached responds to all of your comments and correctly compiles the three testcases that you provided. Two of these have been included in the original testcase and the third appears separately. Bootstrapped and reg test

Re: [C++ PATCH] Don't ICE on internal calls in check_noexcept_r (PR sanitizer/64984)

2015-02-10 Thread Jason Merrill
OK. Jason

Re: [PATCH] Implement libffi for AARCH64:ILP32

2015-02-10 Thread Andrew Pinski
On Tue, Feb 10, 2015 at 1:20 PM, Richard Henderson wrote: > On 02/10/2015 11:02 AM, Andrew Pinski wrote: >> +#define FFI_SIZEOF_JAVA_RAW 4 > > Since aarch64 doesn't implement the raw java api, this is irrelevant. Ok. This is what I applied to GCC after testing it (I don't have access to the lib

Re: [testsuite] PATCH: Add check_effective_target_pie

2015-02-10 Thread Rainer Orth
Jeff Law writes: >> Subject: [PATCH 1/5] Add check_effective_target_pie >> >> Hi, >> >> This patch adds check_effective_target_pie to check if the current >> multilib generates PIE by default. >> >> Thanks. >> >> H.J. >> --- >> 2015-01-11 H.J. Lu >> >> * gcc.target/i386/pie.c: New test. >

Re: [PATCH] PR64959: SFINAE in UDLs

2015-02-10 Thread Andrea Azzarone
Hi, Thanks for the review. I updated the patch. 2015-2-9 Andrea Azzarone PR c++/64959 * parser.c (lookup_literal_operator): Return all candidates. (cp_parser_userdef_char_literal): Simplify error handling. (cp_parser_userdef_numeric_literal): Pass tf_warning_or_error.

Add CFLAGS_FOR_TARGET to Ada OS Constant Extraction Process

2015-02-10 Thread Joel Sherrill
Hi When building Ada for RTEMS, you need to pass in the CFLAGS_FOR_TARGET to see OS specific .h files. This was already done for the RTS but needed to be added to the process which extracts value settings from the .h files. This patch is against 4.9. OK to apply to 4.8, 4.9, and head assuming it

[doc, committed] fix typos in extended asm section

2015-02-10 Thread Sandra Loosemore
I've checked in this patch to fix a couple more typos in the extended asm documentation, as noted by David. -Sandra 2015-02-10 Sandra Loosemore gcc/ * doc/extend.texi (Extended Asm): Fix typos. Index: gcc/doc/extend.texi =

Re: [PATCH, rs6000, testsuite] Changes for unaligned vector load/store support on POWER8

2015-02-10 Thread Bill Schmidt
Hi David, Sorry I haven't gotten back to this sooner. I've tried to address your comments about the previous patch. Please let me know if I'm off base. Test results are the same as previously. Thanks, Bill [gcc] 2015-02-10 Bill Schmidt * config/rs6000/rs6000.c (rs6000_option_over

[doc, committed] symbol rename pragmas

2015-02-10 Thread Sandra Loosemore
I've committed the attached patch on behalf of David Wohlferd, who has no commit access. It was originally proposed here: https://gcc.gnu.org/ml/gcc-patches/2014-05/msg01365.html After reviewing that thread, I think the change is conservatively correct, and since it's a partial revert of one

[doc, committed] small grammar error fix

2015-02-10 Thread Sandra Loosemore
This fix was suggested by David Wohlferd here: https://gcc.gnu.org/ml/gcc-patches/2015-02/msg00377.html I've checked it in as obvious. -Sandra 2015-02-10 David Wohlferd Sandra Loosemore gcc/ * doc/extend.texi (Loop-Specific Pragmas): Fix grammar error. 2015-

[doc, committed] tidy docs for -masm=dialect

2015-02-10 Thread Sandra Loosemore
This is another small documentation fix suggested by David Wohlferd here: https://gcc.gnu.org/ml/gcc-patches/2015-02/msg00377.html I don't think adding cross-references is controversial, so I've checked in this version. -Sandra 2015-02-10 David Wohlferd Sandra Loosemore

RE: [PATCH, FT32] initial support

2015-02-10 Thread James Bowman
(Thanks to everyone for the review. I have reworked the submission as suggested. This does build cleanly with "--enable-werror-always") FT32 is a new high performance 32-bit RISC core developed by FTDI for embedded applications. Support for FT32 has already been added to binutils. This patch ad

Re: [patch, libgfortran] Bug 57822 - I/O: "(g0)" wrongly prints "E+0000"

2015-02-10 Thread Jerry DeLisle
On 02/08/2015 05:10 PM, Jerry DeLisle wrote: The attached patch fixes this by checking for the case when we are doing g0 editing and the exponent is 0. Regression tested on X86-64. For the larger kinds, we are on a different code path out of necessity, so we need to address this corner case.

[PATCH] Implement boehm-gc for AARCH64:ILP32

2015-02-10 Thread Andrew Pinski
Hi, This is a simple patch which allows all of the boehm-gc testsuite that is included with GCC to pass for AARCH64:ILP32. Basically we need to set CPP_WORDSZ to 32 and the alignment to 4 so we detect pointers correctly. OK for GCC? Bootstrapped and tested for aarch64-linux-gnu with no regressi

Re: [PATCH 1/2, combine] Try REG_EQUAL for nonzero_bits

2015-02-10 Thread Jeff Law
On 02/09/15 19:19, Thomas Preud'homme wrote: From: Andrew Pinski [mailto:pins...@gmail.com] Sent: Tuesday, February 10, 2015 9:57 AM +#ifdef SHORT_IMMEDIATES_SIGN_EXTEND +/* If MODE has a precision lower than PREC and SRC is a non-negative constant + that would appear negative in MODE, sig

Fix i386 wrong code issues introduced by the local flag fix

2015-02-10 Thread Jan Hubicka
Hi, this segfault in c-c++-common/torture/builtin-arith-overflow-12.c is caused by fact that we split ltrans units separating call to alias of local function and its target. Because LTO partitioning turns aliases in boundaries into normal symbols, we end up checking alias for can_change_signature_

RE: [PATCH 1/2, combine] Try REG_EQUAL for nonzero_bits

2015-02-10 Thread Thomas Preud'homme
> From: Jeff Law [mailto:l...@redhat.com] > Sent: Wednesday, February 11, 2015 2:04 PM > > Given the rs6000 is affected, one could do before/after tests natively > in the gcc farm to ensure that removing that code doesn't change the > generated code across a bootstrap. Wouldn't that only tell whe

Re: [RFC][PR target/39726 P4 regression] match.pd pattern to do type narrowing

2015-02-10 Thread Jeff Law
On 02/03/15 05:23, Joseph Myers wrote: +&& TYPE_PRECISION (TREE_TYPE (@0)) == TYPE_PRECISION (TREE_TYPE (@1)) +&& TYPE_UNSIGNED (TREE_TYPE (@0)) == TYPE_UNSIGNED (TREE_TYPE (@1)) +&& TYPE_PRECISION (type) > TYPE_PRECISION (TREE_TYPE (@0))) + (convert (bit_and (inner

Re: [PATCH 1/2, combine] Try REG_EQUAL for nonzero_bits

2015-02-10 Thread Jeff Law
On 02/10/15 23:42, Thomas Preud'homme wrote: From: Jeff Law [mailto:l...@redhat.com] Sent: Wednesday, February 11, 2015 2:04 PM Given the rs6000 is affected, one could do before/after tests natively in the gcc farm to ensure that removing that code doesn't change the generated code across a boot

RE: [PATCH 1/2, combine] Try REG_EQUAL for nonzero_bits

2015-02-10 Thread Thomas Preud'homme
> From: Jeff Law [mailto:l...@redhat.com] > Sent: Wednesday, February 11, 2015 2:49 PM > > > > Wouldn't that only tell whether the macro can stay undefined for > rs6000? > > MD files for rs6000 could have been tighten since then but not others > > backend's MD files. > It's certainly possible, but

Re: Add CFLAGS_FOR_TARGET to Ada OS Constant Extraction Process

2015-02-10 Thread Arnaud Charlet
> When building Ada for RTEMS, you need to pass in the CFLAGS_FOR_TARGET > to see OS specific .h files. This was already done for the RTS but needed to > be added to the process which extracts value settings from the .h files. > > This patch is against 4.9. OK to apply to 4.8, 4.9, and head assum