[llvm-commits] LLD PATCH: add ELF reader basic skeleton code

[Sean Silva]

You should read http://llvm.org/docs/CodingStandards.html

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.

--Sean Silva

On Wed, Jun 13, 2012 at 5:07 PM, Hemant Kulkarni <khemant at codeaurora.org>wrote:

> Hello Everyone ,****
>
> ** **
>
> 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.****
>
> ** **
>
> The ReaderELF will read an object file and instantiate an ELFObjectFile
> object and fill in the atom subclasses that correspond for ELF.****
>
> ** **
>
> ** **
>
> --****
>
> Hemant Kulkarni****
>
> khemant at codeaurora.org****
>
> Qualcomm Innovation Center, Inc is a member of Code Aurora Forum****
>
> ** **
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120613/6776ee91/attachment.html>