Patchwork [wwwdocs] SH 4.8 changes update

login
register
mail settings
Submitter Gerald Pfeifer
Date Oct. 6, 2012, 3:57 p.m.
Message ID <alpine.LNX.2.00.1210061607190.1878@tuna.site>
Download mbox | patch
Permalink /patch/189744/
State New
Headers show

Comments

Gerald Pfeifer - Oct. 6, 2012, 3:57 p.m.
Hi Oleg,

have you considered also documenting the new __builtin_thread_pointer
and __builtin_set_thread_pointer built-ins?  (I just noticed this by
chance.)

On Thu, 4 Oct 2012, Oleg Endo wrote:
> The atomic options of SH have been changed recently.  The attached patch 
> updates the 4.8 changes.html accordingly, plus some minor wording fixes.

Looking at the list of parameters to -matomic-model=, it appears
that a definition list (<dl>) is better suitable than a regular
list, so I went ahead and made the change below.

Gerald
Oleg Endo - Oct. 6, 2012, 4:48 p.m.
On Sat, 2012-10-06 at 17:57 +0200, Gerald Pfeifer wrote:
> Hi Oleg,
> 
> have you considered also documenting the new __builtin_thread_pointer
> and __builtin_set_thread_pointer built-ins?  (I just noticed this by
> chance.)

Yes, sure.  The documentation and www changes updates will follow soon.

> 
> On Thu, 4 Oct 2012, Oleg Endo wrote:
> > The atomic options of SH have been changed recently.  The attached patch 
> > updates the 4.8 changes.html accordingly, plus some minor wording fixes.
> 
> Looking at the list of parameters to -matomic-model=, it appears
> that a definition list (<dl>) is better suitable than a regular
> list, so I went ahead and made the change below.
> 

Thanks!  Looks way better now.

Cheers,
Oleg

Patch

Index: changes.html
===================================================================
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-4.8/changes.html,v
retrieving revision 1.37
diff -u -3 -p -r1.37 changes.html
--- changes.html	5 Oct 2012 19:30:26 -0000	1.37
+++ changes.html	6 Oct 2012 15:32:05 -0000
@@ -258,30 +258,30 @@  by this change.</p>
       <li>A new option <code>-matomic-model=<i>model</i></code> selects the
       model for the generated atomic sequences.  The following models are
       supported:
-      <ul>
-        <li><code>soft-gusa</code><br/>
+      <dl>
+        <dt><code>soft-gusa</code></dt><dd>
         Software gUSA sequences (SH3* and SH4* only).  On SH4A targets this
         will now also partially utilize the <code>movco.l</code> and
         <code>movli.l</code> instructions.  This is the default when the target
-        is <code>sh3*-*-linux*</code> or <code>sh4*-*-linux*</code>.</li>
+        is <code>sh3*-*-linux*</code> or <code>sh4*-*-linux*</code>.</dd>
 
-        <li><code>hard-llcs</code><br/>
+        <dt><code>hard-llcs</code></dt><dd>
         Hardware <code>movco.l</code> / <code>movli.l</code> sequences
-        (SH4A only).</li>
+        (SH4A only).</dd>
 
-        <li><code>soft-tcb</code><br/>
-        Software thread control block sequences.</li>
+        <dt><code>soft-tcb</code></dt><dd>
+        Software thread control block sequences.</dd>
 
-        <li><code>soft-imask</code><br/>
-        Software interrupt flipping sequences (privileged mode only).  This is
-        the default when the target is <code>sh1*-*-linux*</code> or
-        <code>sh2*-*-linux*</code>.</li>
+        <dt><code>soft-imask</code></dt><dd>
+        Software interrupt flipping sequences (privileged mode only).  This
+        is the default when the target is <code>sh1*-*-linux*</code> or
+        <code>sh2*-*-linux*</code>.</dd>
 
-        <li><code>none</code><br/>
+        <dt><code>none</code></dt><dd>
         Generates function calls to the respective <code>__atomic</code>
-        built-in functions.  This is the default for SH64 targets or when the
-        target is not <code>sh*-*-linux*</code>.</li>
-      </ul></li>
+        built-in functions.  This is the default for SH64 targets or when
+        the target is not <code>sh*-*-linux*</code>.</dd>
+      </dl></li>
 
       <li>The option <code>-msoft-atomic</code> has been deprecated.  It is
       now an alias for <code>-matomic-model=soft-gusa</code>.</li>