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: [wwwdocs] Mention GIMPLE and RTL frontends in changes.html


On Fri, 2017-02-03 at 08:09 +0100, Gerald Pfeifer wrote:
> On Thu, 2 Feb 2017, David Malcolm wrote:
> > This patch to the website moves the section about the selftest
> > suite to
> > the bottom of "Other significant improvements" section, and
> > rewrites it
> > to also cover the GIMPLE and RTL "frontends", and tries to couch
> > these
> > changes in terms of the benefit to the end-user (i.e. a more
> > reliable
> > compiler).
> 
> Index: htdocs/gcc-7/changes.html
> ===================================================================
> +    <p>GCC's already extensive self-test suite has gained some new
> +      capabilities, to further improve the reliability of the
> compiler:</p>
> 
> Would "testsuite" be sufficient here?
> 
> (Per codingconventions.html we use "testsuite" as opposed to "test 
> suite", as I just checked.)

Changed.

> +      <li>GCC now has has an internal unit testing API and a suite
> of tests
> +        for programmatic self-testing of implementation
> subsystems.</li>
> 
> Omit "implementation"?
> 
> Those are genuine questions (where I feel a little stronger about 
> the former than the latter), i.e., while I personally would make 
> those changes, I am not making this a requirement of my review.
> 
> The patch is okay, just consider the above, please.

I tried a few variants here, but in the end I considered your
suggestion as the cleanest; thanks!

Committed

Dave


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