Re: Fix PR rtl-optimization/70007

2016-03-01 Thread Uros Bizjak
Hello! > 2016-03-01 Eric Botcazou > > PR rtl-optimization/70007 > * gcse.c (compute_ld_motion_mems): Tidy up and also invalidate memory > references present in REG_EQUAL notes attached to non-SET patterns. > > > 2016-03-01 Eric Botcazou > > * gcc.target/i386/pr70007.c: New test. The testcas

Re: [C PATCH] Don't print -Waddress comparison warnings for macros (PR c/48778)

2016-03-01 Thread Jeff Law
On 03/01/2016 07:10 AM, Marek Polacek wrote: This PR from 2011 reports that -Waddress prints unhelpful warning when the comparison comes from a macro. Since I've added from_macro_expansion_at, this is easy to circumvent. I'm not so sure we actually want to disable the warning in the case of a m

Re: [PATCH] Fix PR68621

2016-03-01 Thread Jeff Law
On 03/01/2016 09:48 AM, Kumar, Venkataramanan wrote: Hi Richard, As discussed in PR, tried to adjust the test case by initializing array, but looks like for building with -fpic it needs visibility to be set a hidden. The below patch does that. Ok for trunk ? diff --git a/gcc/testsuite/Change

Please include ada-hurd.diff upstream (try2)

2016-03-01 Thread Svante Signell
Hello, As time progresses, the ada-hurd.diff patch has now been living in Debian since gcc-4.6 and is now in gcc-5 and gcc-6. Last try to get it included upstream was in 2012, and 2014, e.g see https://gcc.gnu.org/ml/gcc-patches/2012-06/msg0.html and follow-ups. Maybe the time has come now fo

Re: [PATCH PR69052]Check if loop inv can be propagated into mem ref with additional addr expr canonicalization

2016-03-01 Thread Jeff Law
On 03/01/2016 10:08 AM, Bin.Cheng wrote: On Thu, Feb 25, 2016 at 8:47 AM, Bin.Cheng wrote: On Thu, Feb 25, 2016 at 6:39 AM, Jeff Law wrote: On 02/22/2016 02:22 AM, Bin.Cheng wrote: My only question is why didn't you use FOR_EACH_SUBRTX_VRA from rtl-iter.h to walk the RTX expressions in coll

patch for PR233876

2016-03-01 Thread Vladimir Makarov
The following patch should fix PR https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70025 The patch was successfully bootstrapped and tested on x86/x86-64. Committed as rev. 233876. I'll work on the test tomorrow -- I have no access to s390x right now. Index: ChangeLog ===

Re: C++ PATCH for c++/69995 (wrong value with C++14 constexpr)

2016-03-01 Thread Jason Merrill
On 02/29/2016 09:25 AM, Jason Merrill wrote: The bug here was that we were sharing the CONSTRUCTOR between the value of 'a' and the elements of 'result', so changing 'a' also changed the value of result[0]. Oops. And there were a few other places with the same problem. Tested x86_64-pc-linux-

Re: C++ PATCH for c++/51489 (pointer subtraction in constant expression)

2016-03-01 Thread Jason Merrill
On 12/19/2011 12:56 AM, Jason Merrill wrote: DR 1313 removes the blanket prohibition on pointer subtraction in constant expressions and replaces it with a prohibition on operations with undefined behavior, so this testcase ought to work. It wasn't working because our internal representation of p

C++ PATCH for c++/70036 (ICE with unexpanded pack in requires-clause)

2016-03-01 Thread Jason Merrill
Straightforward change. Tested x86_64-pc-linux-gnu, applying to trunk. commit c11146bdabc5961956adfe00bef8e60657c60fa6 Author: Jason Merrill Date: Tue Mar 1 20:52:24 2016 -0500 PR c++/70036 * parser.c (cp_parser_requires_clause): Call check_for_bare_parameter_packs. diff

Re: [PATCH] PR c/69993: improvements to wording of -Wmisleading-indentation

2016-03-01 Thread Patrick Palka
On Tue, Mar 1, 2016 at 3:29 PM, Patrick Palka wrote: > On Tue, Mar 1, 2016 at 1:51 PM, David Malcolm wrote: >> The wording of our output from -Wmisleading-indentation is rather >> confusing, as noted by Reddit user "sysop073" here: >> >> https://www.reddit.com/r/programming/comments/47pejg/gcc_

Re: RFA: MEP: Add newlib-stdint.h to tm_file.

2016-03-01 Thread DJ Delorie
> Looks good to me. Obviously you'll need appropriate ChangeLogs. OK > with the ChangeLogs added. Done.

Re: patch for PR233876

2016-03-01 Thread Jakub Jelinek
On Tue, Mar 01, 2016 at 08:43:21PM -0500, Vladimir Makarov wrote: > The following patch should fix PR > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70025 > > The patch was successfully bootstrapped and tested on x86/x86-64. > > Committed as rev. 233876. Thanks. > I'll work on the test

Re: [PING 3, PATCH] PR/68089: C++-11: Ingore "alignas(0)".

2016-03-01 Thread Dominik Vogt
On Mon, Jan 04, 2016 at 12:33:21PM +0100, Dominik Vogt wrote: > On Fri, Jan 01, 2016 at 05:53:08PM -0700, Martin Sebor wrote: > > On 12/31/2015 04:50 AM, Dominik Vogt wrote: > > >The attached patch fixes C++-11 handling of "alignas(0)" which > > >should be ignored but currently generates an error m

Re: [PING 3, PATCH] S/390: Reduce accuracy of bessel_6.f90.

2016-03-01 Thread Dominik Vogt
On Mon, Jan 11, 2016 at 03:40:56PM +0100, Dominik Vogt wrote: > Another patch reducing the accuracy required in the bessel_6 test. > gcc/testsuite/ChangeLog > > * gfortran.dg/bessel_6.f90: Reduce accuracy for S/390. > >From 70a35dd6f6bf906d8e5907667ad0f04f981a61ac Mon Sep 17 00:00:00 2001

Re: [PING 2, PATCH] libitm: Introduce target macro TARGET_BEGIN_TRANSACTION_ATTRIBUTE.

2016-03-01 Thread Dominik Vogt
On Mon, Feb 01, 2016 at 02:18:48PM +0100, Dominik Vogt wrote: > The attached patch adds the a target specific attribute via the > new target macro TARGET_BEGIN_TRANSACTION_ATTRIBUTE to the > function begin_transaction(). S/390 uses this to set the > soft-float target attribute which is needed to f

Re: [PING 3, PATCH] S/390: Reduce accuracy of bessel_6.f90.

2016-03-01 Thread Jakub Jelinek
On Wed, Mar 02, 2016 at 07:36:23AM +0100, Dominik Vogt wrote: > > Another patch reducing the accuracy required in the bessel_6 test. > > > gcc/testsuite/ChangeLog > > > > * gfortran.dg/bessel_6.f90: Reduce accuracy for S/390. Ok, thanks. > > > >From 70a35dd6f6bf906d8e5907667ad0f04f981a61ac

Re: patch for PR233876

2016-03-01 Thread Jakub Jelinek
On Wed, Mar 02, 2016 at 07:30:12AM +0100, Jakub Jelinek wrote: > On Tue, Mar 01, 2016 at 08:43:21PM -0500, Vladimir Makarov wrote: > > The following patch should fix PR > > > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70025 > > > > The patch was successfully bootstrapped and tested on x86

Re: [C PATCH] Don't print -Waddress comparison warnings for macros (PR c/48778)

2016-03-01 Thread Marek Polacek
On Tue, Mar 01, 2016 at 04:38:13PM -0700, Jeff Law wrote: > On 03/01/2016 07:10 AM, Marek Polacek wrote: > >This PR from 2011 reports that -Waddress prints unhelpful warning when the > >comparison comes from a macro. Since I've added from_macro_expansion_at, > >this is easy to circumvent. I'm not

Re: [PATCH][ARM] PR target/70008

2016-03-01 Thread Michael Collison
Hi Richard, I think we could incorporate your feedback by changing the predicate on operand 1 to "arm_rhs_operand" which allows "s_register_operand" or "arm_immediate_operand". Everything else in my patch would stay the same including splitting the thumb2 pattern out into it's own insn. I'm t

Re: [PATCH] S/390: Reduce accuracy of bessel_6.f90.

2016-03-01 Thread Andreas Krebbel
On 01/11/2016 03:40 PM, Dominik Vogt wrote: > gcc/testsuite/ChangeLog > > * gfortran.dg/bessel_6.f90: Reduce accuracy for S/390. Applied. Thanks! -Andreas-

Re: [PATCH] S/390: Ada: Enable Stack_Check_Probes.

2016-03-01 Thread Andreas Krebbel
On 03/01/2016 01:01 PM, Dominik Vogt wrote: > gcc/ada/ChangeLog > > * system-linux-s390.adsx: Enable Stack_Check_Probes. > * system-linux-s390.ads: Likewise. Applied. Thanks! -Andreas-

<    1   2