You should read <a href="http://llvm.org/docs/CodingStandards.html">http://llvm.org/docs/CodingStandards.html</a><div><br></div><div>I took a peek at the patch and the most things that stuck out most for me are that LLVM uses 2 spaces for indentation. Also in LLVM, source code is kept under 80 columns wide.</div>
<div><br></div><div>--Sean Silva<br><br><div class="gmail_quote">On Wed, Jun 13, 2012 at 5:07 PM, Hemant Kulkarni <span dir="ltr"><<a href="mailto:khemant@codeaurora.org" target="_blank">khemant@codeaurora.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div lang="EN-US" link="blue" vlink="purple"><div><p class="MsoNormal">Hello Everyone ,<u></u><u></u></p><p class="MsoNormal">
<u></u> <u></u></p><p class="MsoNormal">I am working to get ELF port in LLD linker. I am attaching a patch that fills in basic sub-classes in the ReaderELF.  I request feedback for this code.  I also want suggestions for ways to test this Reader as I notice there is no way right now to test COFF reader as well.<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">The ReaderELF will read an object file and instantiate an ELFObjectFile object and fill in the atom subclasses that correspond for ELF.<u></u><u></u></p><p class="MsoNormal">
<u></u> <u></u></p><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">--<u></u><u></u></p><p class="MsoNormal">Hemant Kulkarni<u></u><u></u></p><p class="MsoNormal"><a href="mailto:khemant@codeaurora.org" target="_blank">khemant@codeaurora.org</a><u></u><u></u></p>
<p class="MsoNormal">Qualcomm Innovation Center, Inc is a member of Code Aurora Forum<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p></div></div><br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div>