[llvm-commits] [PATCH] Convert WritingAnLLVMBackend.html to reST

Sean Silva silvas at purdue.edu
Fri Nov 30 16:07:17 PST 2012


Hi Dmitri, sorry for the review latency. This LGTM. In the future you
can move to post-commit review for these kinds of "mechanical"
changes. I have a rule in Phabricator that automatically picks up on
changes in docs/ (it's about the only thing I use Phabricator for
right now...), so I'll catch anything suspicious eventually.

A word of warning though if you decide to take on LangRef: there are
some constructs in langref that are more difficult to convert than I
expected, so I would recommend making a docs/LangRef/ directory and
incrementally converting each part into its own file, then once
everything has been converted, the HTML version can be replaced with
reST which just links the subdirectory into its toctree.

-- Sean Silva

On Thu, Nov 29, 2012 at 11:16 AM, Dmitri Gribenko <gribozavr at gmail.com> wrote:
> Hello,
>
> The attached patch converts WritingAnLLVMBackend.html to reST.  Please review!
>
> Thanks in advance,
> Dmitri
>
> --
> main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if
> (j){printf("%d\n",i);}}} /*Dmitri Gribenko <gribozavr at gmail.com>*/



More information about the llvm-commits mailing list