From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gcc-patches-return-337089-listarch-gcc-patches=gcc.gnu.org@gcc.gnu.org>
Received: (qmail 24853 invoked by alias); 17 Feb 2013 23:07:49 -0000
Received: (qmail 24791 invoked by uid 22791); 17 Feb 2013 23:07:47 -0000
X-SWARE-Spam-Status: No, hits=-51.6 required=5.0	tests=AWL,BAYES_00,KHOP_THREADED
X-Spam-Check-By: sourceware.org
Received: from ainaz.pair.com (HELO ainaz.pair.com) (209.68.2.66)    by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Sun, 17 Feb 2013 23:07:41 +0000
Received: from [192.168.0.132] (vie-188-118-252-092.dsl.sil.at [188.118.252.92])	by ainaz.pair.com (Postfix) with ESMTPSA id 082853F410;	Sun, 17 Feb 2013 18:07:37 -0500 (EST)
Date: Sun, 17 Feb 2013 23:07:00 -0000
From: Gerald Pfeifer <gerald@pfeifer.com>
To: Igor Zamyatin <izamyatin@gmail.com>
cc: gcc-patches@gcc.gnu.org, Uros Bizjak <ubizjak@gmail.com>
Subject: Re: [GCC 4.8 wwwdocs] PATCH: Mention several user-visible changes for x86
In-Reply-To: <CAKdSQZkm2hfJtVinOseoM9fHyczqXn_OR4mMCW0U+rcz7sgCyA@mail.gmail.com>
Message-ID: <alpine.LNX.2.00.1302180003480.1859@tuna.site>
References: <CAKdSQZkX3p=Lg6i_1HHqAmaUM4kAUHywyNC=gGhpvn4-Diqbtw@mail.gmail.com> <CAFULd4Zmb5VKA7f2kLaujchgC8XqFOpgeai0WGum4m0xbWdNag@mail.gmail.com> <CAKdSQZ=EEdrAk2oYhMxZO90L4zx6YVRs-7ow6874eqp1EF1-VA@mail.gmail.com> <CAFULd4YQXpcwRQ1rns_E1fy-GA4kwSvaQG9gRk6DV6Ocgjn6tQ@mail.gmail.com> <CAKdSQZkm2hfJtVinOseoM9fHyczqXn_OR4mMCW0U+rcz7sgCyA@mail.gmail.com>
MIME-Version: 1.0
Content-Type: TEXT/PLAIN; charset=US-ASCII
X-IsSubscribed: yes
Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm
Precedence: bulk
List-Id: <gcc-patches.gcc.gnu.org>
List-Archive: <http://gcc.gnu.org/ml/gcc-patches/>
List-Post: <mailto:gcc-patches@gcc.gnu.org>
List-Help: <mailto:gcc-patches-help@gcc.gnu.org>
Sender: gcc-patches-owner@gcc.gnu.org
X-SW-Source: 2013-02/txt/msg00830.txt.bz2

On Fri, 15 Feb 2013, Igor Zamyatin wrote:
> Is it ok for wwwdocs?

Index: htdocs/gcc-4.8/changes.html
===================================================================
+     <li>Support for the new Intel processor codename Broadwell with RDSEED,
+     ADCX, ADOX, PREFETCHW is available through <code>-madx</code>,
+     <code>-mprfchw</code>, <code>-mrdseed</code>.

Can you make this <code>RDSEED</code>, <code>... and so forth?
(This is a bit borderline, in that one could also see this as
more general references, but usually we mark those up.)

And "...through the ... command-line options."?

+     <li> Support for Intel RTM and HLE intrinsics, built-in

"the ... intrinsics" (and same below for "instruction sets")

+     <li> x86 backend was improved to allow option
<code>-fscedule-insns</code> to work reliably.

"The x86 backend has been improved..."

+     This option can be used to schedule instructions better and can
lead to improved performace in certain cases.

This line is quite long, can you break lines around 76 columns?

And, let's be a bit more brave and omit either "can" or "in certain
cases".  Otherwise this may sounds too unlikely. :-)

The patch is fine with changes along the lines described above.

Thanks,
Gerald