[PINGv2][PATCH] ASan phase place change

2015-03-11 Thread Marat Zakirov
On 03/03/2015 10:36 AM, Marat Zakirov wrote: On 02/24/2015 03:49 PM, Marat Zakirov wrote: On 02/20/2015 03:07 PM, Jakub Jelinek wrote: On Fri, Feb 20, 2015 at 02:59:51PM +0300, Marat Zakirov wrote: Here is simple patch that moves asan phase just behind sanopt for all O0/O1/O2/O3 modes which

Re: [PINGv2][PATCH] ASan phase place change

2015-03-11 Thread Yury Gribov
On 03/11/2015 10:43 AM, Marat Zakirov wrote: On 03/03/2015 10:36 AM, Marat Zakirov wrote: On 02/24/2015 03:49 PM, Marat Zakirov wrote: On 02/20/2015 03:07 PM, Jakub Jelinek wrote: On Fri, Feb 20, 2015 at 02:59:51PM +0300, Marat Zakirov wrote: Here is simple patch that moves asan phase just

Re: [PATCH] Backport ubsan fix to 4.9

2015-03-11 Thread Richard Biener
On Tue, 10 Mar 2015, Marek Polacek wrote: > I'd like to backport the following patch that suppresses bogus ubsan errors. > I had to tweak the testcase a bit since 4.9 doesn't know > -fno-sanitize-recover. > > Bootstrapped/regtested on x86_64-linux, ok for 4.9? Ok. Thanks, Richard. > 2015-03-1

Re: [PATCH ] IRA: Speed up setup_left_conflict_sizes_p

2015-03-11 Thread Richard Biener
On Wed, Mar 11, 2015 at 3:39 AM, Zhouyi Zhou wrote: > From: Zhouyi Zhou > > In function setup_left_conflict_sizes_p, left conflict subnodes sizes > are computed in a down-to-up fashion through the regnodes foreast. > > Speed up the process from prevent node itself to involve in this > computation

Re: [patch] Optimize empty class copies within a C++ return statement

2015-03-11 Thread Jason Merrill
On 03/10/2015 09:45 PM, Aldy Hernandez wrote: + gimplify_expr (&TREE_OPERAND (*expr_p, 0), pre_p, post_p, ... + *expr_p = TREE_OPERAND (*expr_p, 0); Can't you use op0 here, since you're about to throw away the MODIFY_EXPR? OK either way. Jason

[PATCH] Fix typo in tail merging (PR tree-optimization/65388)

2015-03-11 Thread Marek Polacek
This patch fixes a pretty obvious typo: we couldn't possibly want to test whether a variable equals itself. PR c/54979 is an RFE for a warning that would detect such cases. Bootstrapped/regtested on x86_64-linux, ok for active branches? 2015-03-11 Marek Polacek PR tree-optimization/6

Re: [PATCH, TSAN] Fix a crash in ScopedReport::AddThread

2015-03-11 Thread Jakub Jelinek
On Wed, Mar 11, 2015 at 03:41:27PM +0100, Bernd Edlinger wrote: > Hi Jakub, > > > with my OPC UA Server, I observe a reproducible crash in > ScopedReport::AddThread: tctx==NULL > in "if ((u32)rep_->threads[i]->id == tctx->tid)". > > Apparently, Dmitry has already fixed that in the obvious way. >

Re: [PATCH] PR target/65242, Fix powerpc abort in gen_add2_insn

2015-03-11 Thread David Edelsohn
On Mon, Mar 9, 2015 at 7:30 PM, Michael Meissner wrote: > This bug was one I unfortunately introduced with the -mupper-regs support. If > the reload pass needed to reload a PLUS operation (for example, due to using > odd address with the LD/STD instructions), it would go through all of the > regi

Re: [PATCH, TSAN] Fix a crash in ScopedReport::AddThread

2015-03-11 Thread Dmitry Vyukov
OK On Wed, Mar 11, 2015 at 5:41 PM, Bernd Edlinger wrote: > Hi Jakub, > > > with my OPC UA Server, I observe a reproducible crash in > ScopedReport::AddThread: tctx==NULL > in "if ((u32)rep_->threads[i]->id == tctx->tid)". > > Apparently, Dmitry has already fixed that in the obvious way. > > So w

[PATCH] Fix regression caused by PR65310 fix

2015-03-11 Thread Richard Biener
This fixes a vectorizer testcase regression on powerpc where SRA drops alignment info unnecessarily. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2015-03-11 Richard Biener PR tree-optimization/65310 * tree-sra.c (build_ref_for_offset): Also preserve

Re: [PATCH] Fix thunk expansion (PR ipa/64896)

2015-03-11 Thread Yvan Roux
Hi, >>> PR ipa/65236 >>> * cgraphunit.c (cgraph_node::expand_thunk): Enable return slot >>> opt. > > This bugfix adds ipa-icf-6.C test which failed on 4.9 branch as > ipa-icf is not backported on that branch. Is the bugfix still > relevant and we can dropped the testcase

Re: [patch] disable libmpx x32 multilib builds

2015-03-11 Thread Ilya Enkovich
2015-03-11 18:59 GMT+03:00 H.J. Lu : > On Wed, Mar 11, 2015 at 7:37 AM, Matthias Klose wrote: >> current trunk fails to build on x86*-linux, when configured for x32 multilibs >> because libmpx doesn't support these. Disable them. >> >> ok for the trunk? >> >> * Disable libmpx x32 multilib

Re: [patch] disable libmpx x32 multilib builds

2015-03-11 Thread H.J. Lu
On Wed, Mar 11, 2015 at 7:37 AM, Matthias Klose wrote: > current trunk fails to build on x86*-linux, when configured for x32 multilibs > because libmpx doesn't support these. Disable them. > > ok for the trunk? > > * Disable libmpx x32 multilib builds. > > --- a/config-ml.in > +++ b/config

Re: [PATCH/AARCH64] Add missing definition of crypto instruction on cortex-a57.md

2015-03-11 Thread James Greenhalgh
On Wed, Mar 11, 2015 at 04:24:07PM +, Ramana Radhakrishnan wrote: > > > > > Attached patch as text. > > > > 2015-03-11 Junmo Park > > > > * config/arm/cortex-a57.md (cortex_a57_crypto_simple): Add > > crypto_sha256_fast. > > (cortex_a57_crypto_complex): Add crypto

Re: [PATCH] PR target/65242, Fix powerpc abort in gen_add2_insn

2015-03-11 Thread David Edelsohn
Jeff, I completely agree. The example exposed a problematic alternative in the pattern and I would like to fix a latent problem, in addition to Mike's patch. Thanks, David On Wed, Mar 11, 2015 at 12:05 PM, Jeff Law wrote: > On 03/11/15 08:44, David Edelsohn wrote: >> >> On Mon, Mar 9, 2015 at

Re: [PATCH] PR target/65240, Fix Power{7,8} insn constraint issue with -O3 -ffast-math

2015-03-11 Thread David Edelsohn
On Thu, Mar 5, 2015 at 3:06 PM, Michael Meissner wrote: > This patch fixes PR 65240, which was a latent bug that was introduced when I > added the -mupper-regs support to the PowerPC compiler. In the PowerPC > compiler, if you use -ffast-math, the compiler allows floating point constants > in mov

[wwwdocs] Document __has_attribute in /gcc-5/changes.html

2015-03-11 Thread Jonathan Wakely
This is a slightly-modified version of Ed's patch from https://gcc.gnu.org/ml/gcc-patches/2015-01/msg02357.html which I plan to commit to CVS tomorrow. Index: changes.html === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-5/changes.html,v re

Re: [wwwdocs] Document __has_attribute in /gcc-5/changes.html

2015-03-11 Thread Jakub Jelinek
On Wed, Mar 11, 2015 at 06:09:11PM +, Jonathan Wakely wrote: > This is a slightly-modified version of Ed's patch from > https://gcc.gnu.org/ml/gcc-patches/2015-01/msg02357.html which I plan > to commit to CVS tomorrow. > > Index: changes.html >

Re: [patch,avr]: Part3 and 4: Fix various problems with specs and specs file generation.

2015-03-11 Thread Denis Chertykov
2015-03-10 18:02 GMT+03:00 Georg-Johann Lay : > This is just a small addendum to the option and specs handling: > > - Document new avr-gcc command options > > - Change -march= to -mmcu= in some test cases > > - Add comfigure test to detect whether gas supports -mrmw and --mlink-relax. > > - Use res

Re: [wwwdocs] Document __has_attribute in /gcc-5/changes.html

2015-03-11 Thread Jonathan Wakely
On 11/03/15 19:14 +0100, Jakub Jelinek wrote: On Wed, Mar 11, 2015 at 06:09:11PM +, Jonathan Wakely wrote: This is a slightly-modified version of Ed's patch from https://gcc.gnu.org/ml/gcc-patches/2015-01/msg02357.html which I plan to commit to CVS tomorrow. Index: changes.html =

[PATCH] Fix some memory leaks

2015-03-11 Thread Jakub Jelinek
Hi! Valgrind reported some memory leaks. record_builtin_type calls just get_identifier on the name, and get_identifier never uses the original string for storage, it always allocates it on its own, so using xstrdup as get_identifier argument leaks the memory. Bootstrapped/regtested on x86_64-lin

[PATCH] Fix __has_{cpp_}attribute with -traditional-cpp (PR preprocessor/65238)

2015-03-11 Thread Jakub Jelinek
Hi! __has_{cpp_,}attribute builtin macros are effectively function-like macros taking one argument (and the ISO preprocessor expands macros in the argument which is IMHO desirable), but the traditional preprocessor has been crashing on them or reporting errors. As the hook uses cpp_get_token and t

Re: [PATCH] Fix some memory leaks

2015-03-11 Thread Richard Biener
On March 11, 2015 8:04:25 PM CET, Jakub Jelinek wrote: >Hi! > >Valgrind reported some memory leaks. record_builtin_type calls >just get_identifier on the name, and get_identifier never uses the >original >string for storage, it always allocates it on its own, so using xstrdup >as get_identifier a

Re: [PATCH] Fix typo in tail merging (PR tree-optimization/65388)

2015-03-11 Thread Richard Biener
On March 11, 2015 2:32:18 PM CET, Marek Polacek wrote: >This patch fixes a pretty obvious typo: we couldn't possibly want to >test >whether a variable equals itself. PR c/54979 is an RFE for a warning >that >would detect such cases. > >Bootstrapped/regtested on x86_64-linux, ok for active branche

Re: [PATCH] optionally disable global check

2015-03-11 Thread Jakub Jelinek
On Fri, Mar 06, 2015 at 05:23:47PM +0300, Marat Zakirov wrote: > Hi all! > > Currently !ASAN_GLOBALS disables red-zones for global variables but keeps > their checks. This simple patch disables these checks too. I've noticed UNRESOLVED: c-c++-common/asan/no-asan-check-glob.c -O0 scan-tree-dum

Re: [C++ Patch] PR 65323

2015-03-11 Thread Jason Merrill
On 03/06/2015 03:36 AM, Paolo Carlini wrote: this is a regression about duplicate warnings with -Wzero-as-null-pointer-constant. The regression is rather old, affects 4_8-branch too, and started when check_default_argument got a perform_implicit_conversion_flags call which warns a first time, the

[PATCH][RFA] [PR rtl-optimization/64317] Enhance postreload-gcse.c to eliminate more redundant loads

2015-03-11 Thread Jeff Law
64317 is a P2 regression related to making the PIC register a pseudo register on the x86 port. Basically post-LRA we still have some redundant loads of the PIC register from memory (it's spilled to memory as other objects are more profitable to keep in registers). But of the set of ~10 actu

No libstdc++ for nvptx

2015-03-11 Thread Thomas Schwinge
Hi! I committed the following in r221362: commit 5121139d0e8b22290fc9e752b07752d8a1b70182 Author: tschwinge Date: Wed Mar 11 21:42:50 2015 + No libstdc++ for nvptx. The C++ front end insists to link against libstdc++ -- which we don't build: $ < build-gcc/gcc/tes

Re: RFC: Building a minimal libgfortran for nvptx

2015-03-11 Thread Thomas Schwinge
Hi! On Fri, 14 Nov 2014 18:08:32 +0100, Bernd Schmidt wrote: > New patch below, [...] ... got committed. I now committed the following in r221363: commit 83ba0e65833dd081db921f8c2b3277316590753c Author: tschwinge Date: Wed Mar 11 21:42:56 2015 + libgfortran LIBGFOR_MINIMAL enhance

Re: [PATCH] PR target/65240, Fix Power{7,8} insn constraint issue with -O3 -ffast-math

2015-03-11 Thread Michael Meissner
On Wed, Mar 11, 2015 at 01:02:06PM -0400, David Edelsohn wrote: > I am concerned with the create_TOC_reference use for TARGET_TOC. Has > this been tested with big endian -mcmodel=small? Yes, that was a problem. Patch coming up soon. Thanks. -- Michael Meissner, IBM IBM, M/S 2506R, 550 King St

Re: [PATCH] PR target/65240, Fix Power{7,8} insn constraint issue with -O3 -ffast-math

2015-03-11 Thread David Edelsohn
On Wed, Mar 11, 2015 at 6:21 PM, Michael Meissner wrote: > On Wed, Mar 11, 2015 at 01:02:06PM -0400, David Edelsohn wrote: >> I am concerned with the create_TOC_reference use for TARGET_TOC. Has >> this been tested with big endian -mcmodel=small? > > Yes, that was a problem. Patch coming up soon

[PATCH V2] IRA: Speed up setup_left_conflict_sizes_p

2015-03-11 Thread Zhouyi Zhou
From: Zhouyi Zhou In function setup_left_conflict_sizes_p, left conflict subnodes sizes are computed in a bottom-to-up fashion through the regnodes foreast. Speed up the process from prevent node itself to involve in this computation. I have no write access to GCC SVN repository, if it OK, ca

[testsuite, committed] require fpic support for g++.dg/ipa/pr64146.C

2015-03-11 Thread Sandra Loosemore
I noticed that g++.dg/ipa/pr64146.C was FAILing on nios2-elf because this target doesn't support -fpic; the ABI defines the PIC relocations for GNU/Linux only, so -fpic gives a "sorry" in the bare-metal toolchain. From reading the PR, my understanding is that the bug is specific to PIC and the