Re: [PATCH 8a/9] Introduce class function_reader (v7)

On 12/02/2016 07:44 PM, David Malcolm wrote: The two flag assignments don't seem to be needed; I think this is due to adding: if (node->native_rtl_p ()) node->force_output = 1; to cgraph_node::finalize_function in patch 9. Should I lose them (and the comment)? Let's keep this patch

Re: [PATCH GCC]Simplify (cond (cmp (convert? x) c1) (op x c2) c3) -> (op (minmax x c1) c2)

On Wed, Nov 30, 2016 at 3:10 PM, Richard Biener wrote: > On Fri, Nov 18, 2016 at 5:53 PM, Bin Cheng wrote: >> Hi, >> This is a rework of https://gcc.gnu.org/ml/gcc-patches/2016-10/msg02007.html. >> Though review comments suggested it could be merged

[PR middle-end/78328] [committed] wrong wording for unbounded alloca case

The problem here is that as we follow the cast from an unsigned int to __SIZE_TYPE__, we ignore the VR_ANTI_RANGE of 7 exhibited by the test in the PR: +void g (unsigned int n) +{ + if (n == 7) +n = 11; + f (__builtin_alloca (n)); +} Since we can't get any meaningful information from

Re: [PING] (v2) Add a "compact" mode to print_rtx_function

On Thu, Dec 01, 2016 at 01:27:55PM +0100, Bernd Schmidt wrote: > On 12/01/2016 11:12 AM, Dominik Vogt wrote: ... > >I'd like to get our test failure fixed, either by changing > >print-rtl.c or our test case. Is the above patch good for trunk? > >It does fix the s390 test failure. > > I still

Re: [PATCH] PR target/78639, Fix code generation on Power9

On Thu, Dec 01, 2016 at 07:46:31PM -0500, Michael Meissner wrote: > The previous code before 242679 used 'wY', but I deleted the 'w' by accident. > > This bug showed up in compiling PUGHReduce/Reduction.c in the cactusADM Spec > 2006 benchmark suite. I have verified that this fixes the problem.

[Committed,fortran] Plug memory leak.

While looking at code related to PR fortran/78618, I noticed a memory leak. The following patch plus the leak. 2016-12-02 Steven G. Kargl * simplify.c (gfc_convert_char_constant): Free result on error. Index: simplify.c

Re: [RFA] Handle target with no length attributes sanely in bb-reorder.c

On Fri, Dec 02, 2016 at 09:47:00AM +0100, Richard Biener wrote: > >> STC tries to make as large as possible consecutive "traces", mainly to > >> help with instruction cache utilization and hit rate etc. It cannot do > >> a very good job if it isn't allowed to copy blocks. > >> > >> "simple" tries

Re: [PATCH] Turn on gnu-indirect-function by default on PowerPC servers

On Fri, 2 Dec 2016, Michael Meissner wrote: > This patch enables --enable-gnu-indirect-function on all PowerPC linux > systems, > except for targetting Android or uclib. It is exactly the same code as we use > in the i[34567]86-*-linux, x86_64-*-linux*, s390-*-linux*, and s390x-*-linux* >

<    1   2