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] fix PR 44782, implement -fmax-errors as a common option

From: Tobias Burnus <burnus at net-b dot de>
To: Chris Lattner <clattner at apple dot com>
Cc: Nathan Froyd <froydnj at codesourcery dot com>, "Joseph S. Myers" <joseph at codesourcery dot com>, gcc-patches at gcc dot gnu dot org, gdr at integrable-solutions dot net, fortran at gcc dot gnu dot org
Date: Tue, 09 Nov 2010 20:03:07 +0100
Subject: Re: [PATCH] fix PR 44782, implement -fmax-errors as a common option
References: <20101109141929.GB7991@nightcrawler> <Pine.LNX.4.64.1011091525450.981@digraph.polyomino.org.uk> <20101109164437.GE7991@nightcrawler> <BF8EC886-0B57-445B-A5CE-E57B7CD39C46@apple.com> <4CD9979B.4010709@net-b.de> <030F4D04-79E2-47BF-A198-3FBD41D8A911@apple.com>

Chris Lattner wrote:

That's fine, and perhaps also a good idea. However, there are lots of already shipped versions of clang out there. They all support -ferror-limit but not -fmax-error. IMO, -ferror-limit is a better name anyway, why not enhance gfortran to accept -ferror-limit?

Ditto for gfortran. Regarding the name: -fmax-errors=<n> (with "-s") and -ferror-limit=<n> sound both fine to me - I do not find the limit version superior. Regarding the accepting the other option additionally, that's what Nathan already proposed:

> On Tue, 9 Nov 2010, Nathan Froyd wrote:

 I suppose if people wanted, we could add -ferror-limit as an undocumented
 alias.

Tobias

References:
- [PATCH] fix PR 44782, implement -fmax-errors for C-family languages
  - From: Nathan Froyd
- Re: [PATCH] fix PR 44782, implement -fmax-errors for C-family languages
  - From: Joseph S. Myers
- Re: [PATCH] fix PR 44782, implement -fmax-errors as a common option
  - From: Nathan Froyd
- Re: [PATCH] fix PR 44782, implement -fmax-errors as a common option
  - From: Chris Lattner
- Re: [PATCH] fix PR 44782, implement -fmax-errors as a common option
  - From: Tobias Burnus
- Re: [PATCH] fix PR 44782, implement -fmax-errors as a common option
  - From: Chris Lattner

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