[LLVMbugs] [PATCH] Clean MachineFrameInfo docs

Reid Spencer reid at x10sys.com
Tue May 23 08:29:19 PDT 2006


Looks fine to me. However, would you add something like:

/// @brief Abstract Stack Frame Information Class

to the bottom of the comment block? This just helps summarize the class
at the top of the generated HTML page.

Thanks,

Reid.

On Tue, 2006-05-23 at 17:48 +0400, Vladimir Prus wrote:
> The attached patch moves documented for MachineFrameInfo right before the 
> class, makes it a Doxygen comment and line-wraps it to 80 symbols. Now the 
> doxygen documentation for that class really says something.
> 
> Ok to commit?
> 
> - Volodya
> _______________________________________________
> LLVMbugs mailing list
> LLVMbugs at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvmbugs
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
URL: <http://lists.llvm.org/pipermail/llvm-bugs/attachments/20060523/33e4282d/attachment.sig>


More information about the llvm-bugs mailing list