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 d...@ubuntu.com 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 +++

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] 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 meiss...@linux.vnet.ibm.com 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

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 bernd.edlin...@hotmail.de 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

[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 rguent...@suse.de PR tree-optimization/65310 * tree-sra.c

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 l...@redhat.com wrote: On 03/11/15 08:44, David Edelsohn wrote: On Mon, Mar

Re: [patch] disable libmpx x32 multilib builds

2015-03-11 Thread Ilya Enkovich
2015-03-11 18:59 GMT+03:00 H.J. Lu hjl.to...@gmail.com: On Wed, Mar 11, 2015 at 7:37 AM, Matthias Klose d...@ubuntu.com 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? *

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 junmoz.p...@samsung.com * config/arm/cortex-a57.md (cortex_a57_crypto_simple): Add crypto_sha256_fast. (cortex_a57_crypto_complex): Add

[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

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,

No libstdc++ for nvptx

2015-03-11 Thread Thomas Schwinge
Hi! I committed the following in r221362: commit 5121139d0e8b22290fc9e752b07752d8a1b70182 Author: tschwinge tschwinge@138bc75d-0d04-0410-961f-82ee72b054a4 Date: Wed Mar 11 21:42:50 2015 + No libstdc++ for nvptx. The C++ front end insists to link against libstdc++ -- which we

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 ber...@codesourcery.com wrote: New patch below, [...] ... got committed. I now committed the following in r221363: commit 83ba0e65833dd081db921f8c2b3277316590753c Author: tschwinge tschwinge@138bc75d-0d04-0410-961f-82ee72b054a4 Date:

[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

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 a...@gjlay.de: 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. -

[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

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

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

Re: [PATCH] Fix some memory leaks

2015-03-11 Thread Richard Biener
On March 11, 2015 8:04:25 PM CET, Jakub Jelinek ja...@redhat.com 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

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

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] 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 meiss...@linux.vnet.ibm.com 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.

[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

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 zhouzho...@gmail.com wrote: From: Zhouyi Zhou yizhouz...@ict.ac.cn 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

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-10

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

[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 pola...@redhat.com PR

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 V2] IRA: Speed up setup_left_conflict_sizes_p

2015-03-11 Thread Zhouyi Zhou
From: Zhouyi Zhou zhouzho...@gmail.com 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

[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

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

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. So we

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 meiss...@linux.vnet.ibm.com 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