Hi Duncan, On Fri, 2007-03-30 at 16:39 +0200, Duncan Sands wrote: > Hi Reid, > > > > - possible, by <a href="CommandGuide/html/bugpoint.html">bugpoint</a> > > > or > > > + possible, by <a href="Bugpoint.html">bugpoint</a> or > > > > These aren't the same document. If the bugpoint.html document isn't > > there its an artifact of not generating the HTML version from the POD > > version. This isn't a huge deal for the link, but do we have a larger > > issue generating the command pages? > > I first changed this mistakenly, because I hadn't generated the html files, > but when I realized it was a mistake it struck me that it was better this way, > so I left it rather than changing it back.
Okay, that's fine. > > > > - <li><a href="#checkpoint">Check Points</a></li> > > > > Can you delete the CheckPoints section too? Its going away soon, when > > PR1289 is closed. > > There is no checkpoints section, there is only "Data Layout". However > "Data Layout" talks only of checkpoints. Would you like me to delete it? > > <div class="doc_subsection"> > <a name="datalayout">Data Layout</a> > </div> > <div class="doc_text"> > <p>At the top level, the keyword <tt>checkpoint</tt> may be used to force > the > assembler to check point the types and values it has defined so far. Any > unresolved types or references will generate error messages. This is useful > as an assertion point or for debugging.</p> > </div> Yes, delete that whole <div class="doc_text"> and the doc_subsection too if that's the only paragraph in it. The checkpoint thing is not supported any more. I know this isn't your change, but since you're in there ... :) > > > > > - <tr><td><tt>void</tt></td><td>No value</td></tr> > > > + <tr><td><tt><a name="t_void">void</a></tt></td><td>No > > > value</td></tr> > > > <tr><td><tt>i8</tt></td><td>8-bit value</td></tr> > > > <tr><td><tt>i32</tt></td><td>32-bit value</td></tr> > > > <tr><td><tt>float</tt></td><td>32-bit floating point > > > value</td></tr> > > > > Why not put in links for the other types? > > Because they have their own sections with an anchor, eg t_integer refers to > the > section on integers. Void is the only type that doesn't have its own section. > Probably it should get one. Oh, yeah, sorry, thought that was an href="#t_void" you put in there. In this case, this link isn't good. We don't want a link coming to this table because itw ill confuse the reader. Please add a section near t_integer for t_void. Then, make all of these table entries have links to the type's section. Thx, Reid. > > Ciao, > > Duncan. _______________________________________________ llvm-commits mailing list llvm-commits@cs.uiuc.edu http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
