<div dir="ltr">On Fri, Jun 15, 2018 at 9:31 AM, Ellis Hoag <span dir="ltr"><<a href="mailto:ellis.sparky.hoag@gmail.com" target="_blank">ellis.sparky.hoag@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">I wanted to add CodeView assembly directives to the docs for reference. The documentation was taken from comments at lib/MC/MCParser/AsmParser.cpp<br></div>
</blockquote></div><br>Thanks for the patch!</div><div class="gmail_extra"><br></div><div class="gmail_extra">I think that it would be nice if you could make it explicit that the parameters are parameters.  Right now, they will be rendered as fixed width content.</div><div class="gmail_extra"><br></div><div class="gmail_extra">Please use fixed width text for the references to the directives in the documentation (e.g. ``.cv_loc`` in the ``.cv_inline_site_id`` documentation, ``.file``/``.loc`` in the ``.cv_loc`` documentation).<br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature">Saleem Abdulrasool<br>compnerd (at) compnerd (dot) org</div>
</div></div>