[lld] r228905 - [ELF] Add LinkingContext to the ELFReader.

[Chandler Carruth]

On Thu, Feb 12, 2015 at 12:06 PM, Rui Ueyama <ruiu at google.com> wrote:

> This is a large patch that possibly many places in ELF. Since many people
> are interested in LLD ELF support and code health of LLD in general,
> please send this kind of patch to review before submitting in the future.
> Thanks.
>

Another thing to consider -- discuss these kinds of layering and API design
issues first, even without a patch.

Sometimes the patch is obvious and uninteresting. It can be a lot of work
to build the patch, mail it, etc etc. But the idea of the API change can be
explained easily and considered.

I actually do a lot of this on IRC to reduce the surprise of new APIs while
having fast response times. But email usually works well too.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150212/e8207956/attachment.html>