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: [GCC 4.8 wwwdocs] PATCH: Mention several user-visible changes for x86


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


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