[PATCH] Fix JUMP_LABEL documentation

On Tue, Sep 20, 2016 at 12:11:45PM -0500, Segher Boessenkool wrote: > I'm just pointing out the documentation is out-of-date here. I'll do > a patch. Like this. Okay for trunk, 6, 5? Thanks, Segher 2016-09-20 Segher Boessenkool * doc/rtl.texi (JUMP_LABEL): Document RETURN and SI

Re: Register numbers in RTL dumps (was Re: [PATCH 0/9] RFC: selftests based on RTL dumps)

On Tue, 2016-09-20 at 09:20 -0600, Jeff Law wrote: > On 09/20/2016 08:34 AM, Bernd Schmidt wrote: > > On 09/20/2016 04:32 PM, David Malcolm wrote: > > > > > > To summarize so far: you want every pseudo to have its regno > > > dumped > > > with a 'p' prefix, and renumber them on dump (and then on l

Re: [gofrontend-dev] Go patch committed: Avoid warning by using a local var for std::ofstream

* Ian Lance Taylor: > GCC PR 77625 is about a warning while compiling the Go frontend. The > Go frontend called `new std::ofstream()`, and the warning is "error: > ‘new’ of type ‘std::ofstream {aka std::basic_ofstream}’ with > extended alignment 16". Frankly I'm not sure how this supposed to > w

Re: [PATCH] - improve sprintf buffer overflow detection (middle-end/49905)

+ /* The minimum and maximum length. The MAXLEN pointer stays unchanged + but MINLEN may be cleared during the execution of the function. */ + tree *minlen = length; + tree* const maxlen = length + 1; Check formatting here. I'm not sure if the formatting standards explicitly state how t

Re: [PATCH] - improve sprintf buffer overflow detection (middle-end/49905)

On Tue, 2016-09-20 at 13:35 -0600, Martin Sebor wrote: [...] > > > +#if 0 > ... > > > +#else > ... > > > +#endif > > Please remove the #if 0'd code. > > Sorry about that. This was left over from my effort to persuade > the substring_loc class to underline just the last quote of the > format str

[Patch, fortran] PR77657 - link error with implementation of user-defined derived type input/output (UD-DTIO) in child extending abstract parent

Dear All, Once found, the fix for this PR is trivial. The generic name is only to be found in the parent derived type. Since this is over-ridden in the type extension, not only is the wrong symbol selected for the dtio procedure but, being abstract, the procedure does not exist. The mechanism is b

Re: Document OpenACC status for GCC 6

[ Old e-mail alert ] On Fri, 22 Apr 2016, Thomas Schwinge wrote: > Thanks for the review; OK to commit as follows? And then, should > something be added to the "News" section on > itself, too? (I don't know the policy for that. We didn't suggest that > for GCC 5, because

Re: [PATCH] Optimise the fpclassify builtin to perform integer operations when possible

On Tue, Sep 20, 2016 at 01:19:07PM +0100, Tamar Christina wrote: > On 19/09/16 23:16, Michael Meissner wrote: > >On Mon, Sep 12, 2016 at 04:19:32PM +, Tamar Christina wrote: > >>Hi All, > >> > >>This patch adds an optimized route to the fpclassify builtin > >>for floating point numbers which ar

Re: [Patch, fortran] PR77657 - link error with implementation of user-defined derived type input/output (UD-DTIO) in child extending abstract parent

On 09/20/2016 01:15 PM, Paul Richard Thomas wrote: > Dear All, > > Once found, the fix for this PR is trivial. The generic name is only > to be found in the parent derived type. Since this is over-ridden in > the type extension, not only is the wrong symbol selected for the dtio > procedure but, b

[PATCH] PR68212, Correct frequencies/counts when unrolling

The following patch corrects frequency/count values computed when generating the switch blocks/peeled loop copies before the loop. The two main problem areas were for the peeled copies duplicate_loop_to_header_edge was not using the preheader frequency as part of the scale factor when peeling a

RE: [PATCH] Fix PR tree-optimization/77654

Add missing attachment. Doug gcc/ PR tree-optimization/77654 * tree-ssa-alias.c (issue_prefetch_ref): Add call to duplicate_ssa_name_ptr_info. From ec9069b7b7b07d5fda9c04aaa9b385fba89a6e16 Mon Sep 17 00:00:00 2001 From: Doug Gilmore Date: Tue, 6 Sep 2016 10:18:42 -0700 Su

RE: [PATCH] Fix PR tree-optimization/77654

It looks like the original message was dropped, resending. Doug From: Doug Gilmore Sent: Tuesday, September 20, 2016 2:12 PM To: gcc-patches@gcc.gnu.org; rgue...@gcc.gnu.org Subject: [PATCH] Fix PR tree-optimization/77654 From: https://gcc.gnu.org/bugzill

Re: [PATCHv3, resent] Add a warning for suspicious use of conditional expressions in boolean context

On Tue, Sep 20, 2016 at 05:30:36PM +0200, Bernd Schmidt wrote: > On 09/20/2016 05:18 PM, Jeff Law wrote: > > > I assume HARD_FRAME_POINTER_REGNUM is never zero. > > It could be zero. It's just a hard register number. No target has the > > property that its hard frame pointer register is 0 though

Re: [PATCHv3, resent] Add a warning for suspicious use of conditional expressions in boolean context

On 09/21/2016 01:09 AM, Trevor Saunders wrote: I thought I remember discussing this macro with you, but see what was checked in I'll believe I'm thinking of something similar but different. I think this here was an earlier patch and the one we were discussing recently was the other macro with

[PATCH], PR 77670, Fix PowerPC ISA 3.0 -mpower9-minmax code generation

There are some instructions in PowerPC ISA 3.0 that are not currently enabled by default when you use -mcpu=power9. These instructions are currently added with the -mpower9-minmax switch. I recently built Spec 2006 with the current trunk compiler, and I discovered that three of the benchmarks (ga

Re: [PATCH] Optimise the fpclassify builtin to perform integer operations when possible

On Tue, 20 Sep 2016, Michael Meissner wrote: > It would be better to have a fpclassify2 pattern, and if it isn't > defined, then do the machine independent processing. That is the way it is > done elsewhere. But note: * The __builtin_fpclassify function takes arguments for all the possible FP_

Re: [PATCHv3, resent] Add a warning for suspicious use of conditional expressions in boolean context

On Wed, Sep 21, 2016 at 01:13:41AM +0200, Bernd Schmidt wrote: > On 09/21/2016 01:09 AM, Trevor Saunders wrote: > > > > I thought I remember discussing this macro with you, but see what was > > checked in I'll believe I'm thinking of something similar but different. > > I think this here was an e

Re: [PATCH] - improve sprintf buffer overflow detection (middle-end/49905)

On 09/20/2016 01:38 PM, David Malcolm wrote: On Tue, 2016-09-20 at 13:35 -0600, Martin Sebor wrote: [...] +#if 0 ... +#else ... +#endif Please remove the #if 0'd code. Sorry about that. This was left over from my effort to persuade the substring_loc class to underline just the last quo

Re: [PATCHv3, resent] Add a warning for suspicious use of conditional expressions in boolean context

On 9/20/16, Trevor Saunders wrote: > On Wed, Sep 21, 2016 at 01:13:41AM +0200, Bernd Schmidt wrote: >> On 09/21/2016 01:09 AM, Trevor Saunders wrote: >> > >> > I thought I remember discussing this macro with you, but see what was >> > checked in I'll believe I'm thinking of something similar but >

[Patch, testsuite] Make pr64130.c pass for avr

Hi, For the lower vrp bound to be 2/-2, unsigned ints must be 4 bytes wide. This isn't true for targets like avr. Explicitly using __UINT32_TYPE__ behind a typedef makes the testcase pass for all targets. Committed to trunk. Regards Senthil gcc/testsuite/ChangeLog: 2016-09-21 Senthil Ku

<    1   2