public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [wwwdocs] news/egcs-vcg.html goes HTML 5 (and looks better)
@ 2018-09-02  9:07 Gerald Pfeifer
  2018-09-02  9:53 ` Gerald Pfeifer
  0 siblings, 1 reply; 2+ messages in thread
From: Gerald Pfeifer @ 2018-09-02  9:07 UTC (permalink / raw)
  To: gcc-patches

Use CSS (aligned with our regular documentation) to highlight code
snippets as opposed to direct encoding; don't <center> some blocks.
This makes this page HTML 5 and actually also looks better.

(We probably could simply remove those tables; not sure why Jeff
added them back then, probably for the sake of coloring?)

Committed.

Gerald

Index: news/egcs-vcg.html
===================================================================
RCS file: /cvs/gcc/wwwdocs/htdocs/news/egcs-vcg.html,v
retrieving revision 1.14
diff -u -r1.14 egcs-vcg.html
--- news/egcs-vcg.html	1 Sep 2018 23:42:07 -0000	1.14
+++ news/egcs-vcg.html	2 Sep 2018 09:03:35 -0000
@@ -35,10 +35,10 @@
     <p>First you should get an impression on what the program and the
     gcc changes do.  Take the following small program.</p>
 
-      <table bgcolor="#a0a0a0">
+      <table>
 	<tr>
 	  <td>
-	    <pre>int
+	    <pre class="smallexample">int
 gcd (int v1, int v2)
 {
   int l = v1 &lt; v2 ? v1 : v2;
@@ -74,10 +74,10 @@
     should dump.  E.g., giving GCC the option <code>-da</code> dumps
     the files:</p>
 
-    <table bgcolor="#a0a0a0">
+    <table>
       <tr>
 	<td>
-	  <pre># ../cc1 -O2 -o test.o test.c -da
+	  <pre class="smallexample"># ../cc1 -O2 -o test.o test.c -da
  gcd main
 time in parse: 0.010000
 [... some lines removed ...]
@@ -93,10 +93,10 @@
     <p>These files are kind of hard to read if you are not used to
     RTL.  Example?  This is a part of <code>test.c.lreg</code>:</p>
 
-    <center><table bgcolor="#a0a0a0">
+    <table>
       <tr>
 	<td>
-	  <pre>[... some lines removed ...]
+	  <pre class="smallexample">[... some lines removed ...]
 Basic block 5: first insn 25, last 27.
 
 Registers live at start: 6 7 24 25
@@ -133,7 +133,6 @@
 	  </td>
 	</tr>
       </table>
-    </center>
 
     <p>All the information about the basic blocks and the instructions
     and so on is available but not the the most human friendly
@@ -143,10 +142,10 @@
     If you add the option <code>-dv</code> to your
     commandline you get a handful of extra files:</p>
 
-    <table bgcolor="#a0a0a0">
+    <table>
       <tr>
 	<td>
-	  <pre># ../cc1 -O2 -o test.o test.c -da -dv
+	  <pre class="smallexample"># ../cc1 -O2 -o test.o test.c -da -dv
  gcd main
 time in parse: 0.010000
 [... some lines removed ...]
@@ -162,7 +161,7 @@
     <p>If you view these files using a suitable program, you'll get output
     similar to the following:</p>
 
-    <center><img src="vcg1.png" alt="completely folded graph" width="112" height="41" /></center>
+    <img src="vcg1.png" alt="completely folded graph" width="112" height="41" />
 
     <p>These are nodes representing all the functions in the file.  If
     you expand the nodes you can get a picture like

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [wwwdocs] news/egcs-vcg.html goes HTML 5 (and looks better)
  2018-09-02  9:07 [wwwdocs] news/egcs-vcg.html goes HTML 5 (and looks better) Gerald Pfeifer
@ 2018-09-02  9:53 ` Gerald Pfeifer
  0 siblings, 0 replies; 2+ messages in thread
From: Gerald Pfeifer @ 2018-09-02  9:53 UTC (permalink / raw)
  To: gcc-patches, Jeff Law

On Sun, 2 Sep 2018, Gerald Pfeifer wrote:
> (We probably could simply remove those tables; not sure why Jeff
> added them back then, probably for the sake of coloring?)

Done thusly.

And, yes, without the tables or the patch I just committed to our CSS

  https://gcc.gnu.org/ml/gcc-patches/2018-09/msg00031.html

(a concept which did not exist back then, but really helps with such 
matters), the background of the examples would go across the while 
width, which doesn't look that great.

Gerald

Index: news/egcs-vcg.html
===================================================================
RCS file: /cvs/gcc/wwwdocs/htdocs/news/egcs-vcg.html,v
retrieving revision 1.15
diff -u -r1.15 egcs-vcg.html
--- news/egcs-vcg.html	2 Sep 2018 09:05:50 -0000	1.15
+++ news/egcs-vcg.html	2 Sep 2018 09:09:45 -0000
@@ -35,10 +35,7 @@
     <p>First you should get an impression on what the program and the
     gcc changes do.  Take the following small program.</p>
 
-      <table>
-	<tr>
-	  <td>
-	    <pre class="smallexample">int
+<pre class="smallexample">int
 gcd (int v1, int v2)
 {
   int l = v1 &lt; v2 ? v1 : v2;
@@ -64,20 +61,14 @@
   printf ("gcd(%d, %d) = %d\n", v1, v2, gcd (v1, v2));
   return 0;
 }
-	    </pre>
-	  </td>
-	</tr>
-      </table>
+</pre>
 
     <p>If you want to understand how GCC translates this
     programs you use the <code>-d</code> option to select what GCC
     should dump.  E.g., giving GCC the option <code>-da</code> dumps
     the files:</p>
 
-    <table>
-      <tr>
-	<td>
-	  <pre class="smallexample"># ../cc1 -O2 -o test.o test.c -da
+<pre class="smallexample"># ../cc1 -O2 -o test.o test.c -da
  gcd main
 time in parse: 0.010000
 [... some lines removed ...]
@@ -85,18 +76,13 @@
 test.c.addressof  test.c.cse2  test.c.jump   test.c.regmove  test.c.stack
 test.c.bp         test.c.flow  test.c.jump2  test.c.rtl
 test.c.combine    test.c.gcse  test.c.loop   test.c.sched
-test.c.cse        test.c.greg  test.c.lreg   test.c.sched2</pre>
-	</td>
-      </tr>
-    </table>
+test.c.cse        test.c.greg  test.c.lreg   test.c.sched2
+</pre>
 
     <p>These files are kind of hard to read if you are not used to
     RTL.  Example?  This is a part of <code>test.c.lreg</code>:</p>
 
-    <table>
-      <tr>
-	<td>
-	  <pre class="smallexample">[... some lines removed ...]
+<pre class="smallexample">[... some lines removed ...]
 Basic block 5: first insn 25, last 27.
 
 Registers live at start: 6 7 24 25
@@ -129,10 +115,8 @@
 (insn 12 10 13 (set (reg/v:SI 23)
         (reg/v:SI 24)) 54 {movsi+2} (insn_list 6 (nil))
     (nil))
-[... more lines removed ...]</pre>
-	  </td>
-	</tr>
-      </table>
+[... more lines removed ...]
+</pre>
 
     <p>All the information about the basic blocks and the instructions
     and so on is available but not the the most human friendly
@@ -142,10 +126,7 @@
     If you add the option <code>-dv</code> to your
     commandline you get a handful of extra files:</p>
 
-    <table>
-      <tr>
-	<td>
-	  <pre class="smallexample"># ../cc1 -O2 -o test.o test.c -da -dv
+<pre class="smallexample"># ../cc1 -O2 -o test.o test.c -da -dv
  gcd main
 time in parse: 0.010000
 [... some lines removed ...]
@@ -153,10 +134,8 @@
 test.c.addressof.vcg  test.c.cse2.vcg  test.c.jump.vcg   test.c.regmove.vcg
 test.c.bp.vcg         test.c.flow.vcg  test.c.jump2.vcg  test.c.sched.vcg
 test.c.combine.vcg    test.c.gcse.vcg  test.c.loop.vcg   test.c.sched2.vcg
-test.c.cse.vcg        test.c.greg.vcg  test.c.lreg.vcg   test.c.stack.vcg</pre>
-	</td>
-      </tr>
-    </table>
+test.c.cse.vcg        test.c.greg.vcg  test.c.lreg.vcg   test.c.stack.vcg
+</pre>
 
     <p>If you view these files using a suitable program, you'll get output
     similar to the following:</p>

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2018-09-02  9:53 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-09-02  9:07 [wwwdocs] news/egcs-vcg.html goes HTML 5 (and looks better) Gerald Pfeifer
2018-09-02  9:53 ` Gerald Pfeifer

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).