This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [patch] Add support for #pragma GCC unroll

From: Sandra Loosemore <sandra at codesourcery dot com>
To: Eric Botcazou <ebotcazou at adacore dot com>
Cc: <gcc-patches at gcc dot gnu dot org>, Mike Stump <mikestump at comcast dot net>
Date: Wed, 22 Nov 2017 09:45:57 -0700
Subject: Re: [patch] Add support for #pragma GCC unroll
Authentication-results: sourceware.org; auth=none
References: <3136854.5zjt3GTbhu@polaris> <9d733768-8593-4808-5af3-5252016a1a5d@codesourcery.com> <11451135.uX5RqkD26M@polaris>

On 11/21/2017 03:18 AM, Eric Botcazou wrote:

First of all, the structuring in this section is screwed up.  The
discussion and examples for the previous item (#pragma ivdep) should be
moved inside the @table so that you don't have to introduce another
@table here, just insert another entry into the existing one.


That's also the case for the entire subsection just above, namely "Function
Specific Option Pragmas".  I presume the tables must be merged there too?

That would be a good thing generally, but you don't have to fix that aspart of this patch (since you're not touching that section otherwise).


-Sandra

References:
- [patch] Add support for #pragma GCC unroll
  - From: Eric Botcazou
- Re: [patch] Add support for #pragma GCC unroll
  - From: Sandra Loosemore
- Re: [patch] Add support for #pragma GCC unroll
  - From: Eric Botcazou

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]