diff mbox series

[wwwdocs] projects/cfo.html -- simplify

Message ID alpine.LSU.2.21.1809021519240.6216@anthias.pfeifer.com
State New
Headers show
Series [wwwdocs] projects/cfo.html -- simplify | expand

Commit Message

Gerald Pfeifer Sept. 2, 2018, 1:28 p.m. UTC
This uses a new CSS class *once* instead of attributing most elements
in the table with align="right" and also brings us closer to HTML 5
compliance.

Committed.

Comments

Gerald Pfeifer Sept. 2, 2018, 7:21 p.m. UTC | #1
On Sun, 2 Sep 2018, Gerald Pfeifer wrote:
> This uses a new CSS class *once* instead of attributing most elements
> in the table with align="right" and also brings us closer to HTML 5
> compliance.

And this also takes care of align="center" by using the CSS of the
same name I introduced earlier today.

Gerald

Index: projects/cfo.html
===================================================================
RCS file: /cvs/gcc/wwwdocs/htdocs/projects/cfo.html,v
retrieving revision 1.16
diff -u -r1.16 cfo.html
--- projects/cfo.html	2 Sep 2018 13:26:43 -0000	1.16
+++ projects/cfo.html	2 Sep 2018 19:19:17 -0000
@@ -167,15 +167,15 @@
 <table class="right" border="1">
 <tr>
   <th>&nbsp;</th>
-  <th align="center" colspan="2">Code size save</th>
-  <th align="center" colspan="2">Compilation time multiplier</th>
+  <th class="center" colspan="2">Code size save</th>
+  <th class="center" colspan="2">Compilation time multiplier</th>
 </tr>
 <tr>
   <td>Target</td>
-  <td align="center">arm-elf</td>
-  <td align="center">i686-linux</td>
-  <td align="center">arm-elf</td>
-  <td align="center">i686-linux</td>
+  <td class="center">arm-elf</td>
+  <td class="center">i686-linux</td>
+  <td class="center">arm-elf</td>
+  <td class="center">i686-linux</td>
 </tr>
 <tr>
   <td>Sequence abstraction on RTL</td>
Gerald Pfeifer Sept. 10, 2018, 9:29 p.m. UTC | #2
On Sun, 2 Sep 2018, Gerald Pfeifer wrote:
> On Sun, 2 Sep 2018, Gerald Pfeifer wrote:
>> This uses a new CSS class *once* instead of attributing most elements
>> in the table with align="right" and also brings us closer to HTML 5
>> compliance.
> And this also takes care of align="center" by using the CSS of the
> same name I introduced earlier today.

And this is a final cleanup patch.

Committed.

Gerald

Omit border="1" from the performance table.  Make an empty <th> really so.

Index: projects/cfo.html
===================================================================
RCS file: /cvs/gcc/wwwdocs/htdocs/projects/cfo.html,v
retrieving revision 1.17
diff -u -r1.17 cfo.html
--- projects/cfo.html	2 Sep 2018 19:20:55 -0000	1.17
+++ projects/cfo.html	10 Sep 2018 21:27:51 -0000
@@ -164,9 +164,9 @@
 <a href="http://szeged.github.io/csibe/">CSiBE</a> benchmark with respect
 to the mainline at the last merge (2005-07-11).</p>
 
-<table class="right" border="1">
+<table class="right">
 <tr>
-  <th>&nbsp;</th>
+  <th></th>
   <th class="center" colspan="2">Code size save</th>
   <th class="center" colspan="2">Compilation time multiplier</th>
 </tr>
diff mbox series

Patch

Index: gcc.css
===================================================================
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc.css,v
retrieving revision 1.57
diff -u -r1.57 gcc.css
--- gcc.css	2 Sep 2018 11:39:30 -0000	1.57
+++ gcc.css	2 Sep 2018 13:19:37 -0000
@@ -16,6 +16,7 @@ 
 .smaller  { font-size: 80%; }
 
 .left     { text-align:left; }
+.right    { text-align:right; }
 
 .no-margin-top { margin-top:0; }
 .twocolumns { column-counts:2; -moz-column-count:2; }
Index: projects/cfo.html
===================================================================
RCS file: /cvs/gcc/wwwdocs/htdocs/projects/cfo.html,v
retrieving revision 1.14
diff -u -r1.14 cfo.html
--- projects/cfo.html	1 Sep 2018 23:42:09 -0000	1.14
+++ projects/cfo.html	2 Sep 2018 13:25:52 -0000
@@ -164,7 +164,7 @@ 
 <a href="http://szeged.github.io/csibe/">CSiBE</a> benchmark with respect
 to the mainline at the last merge (2005-07-11).</p>
 
-<table width="50%" border="1">
+<table class="right" border="1">
 <tr>
   <th>&nbsp;</th>
   <th align="center" colspan="2">Code size save</th>
@@ -179,43 +179,43 @@ 
 </tr>
 <tr>
   <td>Sequence abstraction on RTL</td>
-  <td align="right">1.07%</td>
-  <td align="right">1.43%</td>
+  <td>1.07%</td>
+  <td>1.43%</td>
 
-  <td align="right">1.94</td>
-  <td align="right">1.26</td>
+  <td>1.94</td>
+  <td>1.26</td>
 </tr>
 <tr>
   <td>Sequence abstraction on Tree</td>
-  <td align="right">0.34%</td>
-  <td align="right">0.18%</td>
+  <td>0.34%</td>
+  <td>0.18%</td>
 
-  <td align="right">1.25</td>
-  <td align="right">1.25</td>
+  <td>1.25</td>
+  <td>1.25</td>
 </tr>
 <tr>
   <td>Local factoring on RTL</td>
-  <td align="right">0.11%</td>
-  <td align="right">0.19%</td>
+  <td>0.11%</td>
+  <td>0.19%</td>
 
-  <td align="right">1.01</td>
-  <td align="right">0.99</td>
+  <td>1.01</td>
+  <td>0.99</td>
 </tr>
 <tr>
   <td>Local factoring on Tree-SSA</td>
-  <td align="right">0.31%</td>
-  <td align="right">0.24%</td>
+  <td>0.31%</td>
+  <td>0.24%</td>
 
-  <td align="right">1.02</td>
-  <td align="right">1.01</td>
+  <td>1.02</td>
+  <td>1.01</td>
 </tr>
 <tr>
   <td><b>Overall</b></td>
-  <td align="right"><b>1.96%</b></td>
-  <td align="right"><b>1.94%</b></td>
+  <td><b>1.96%</b></td>
+  <td><b>1.94%</b></td>
 
-  <td align="right"><b>2.08</b></td>
-  <td align="right"><b>1.49</b></td>
+  <td><b>2.08</b></td>
+  <td><b>1.49</b></td>
 </tr>
 </table>