[lld] r242088 - Initial ELF port.

Chandler Carruth chandlerc at google.com
Tue Jul 14 11:29:18 PDT 2015


On Tue, Jul 14, 2015 at 11:08 AM Rafael EspĂ­ndola <
rafael.espindola at gmail.com> wrote:

> > Did you discuss this with *anyone* else in the LLD community?
> >
> > Dropping an *enourmous* patch like this without any review or discussion
> is
> > completely unacceptable. Please revert this, and post a patch for review.
> >
> > Even when Rui first did his COFF port, he posted it for review and
> discussed
> > it prior to submitting it.
>
> Sorry, this is in part my fault.
>
> I knew that Davide was working on a new port and during this visit
> California I learned that Michael was doing one too. That was
> overlapping with Davide's work and having merging problems with the
> work I am doing to streamline ELF.h.
>
> Michael had talked with Davide, and I overreacted and asked him to
> commit something to stop the work overlap and conflicts.
>

FWIW, I think sending a review will likely be almost as effective. =]
Naturally, I understand the desire to get folks coordinating on this, I'm
personally super excited about re-thinking the ELF side of things.

>
> Cheers,
> Rafael
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150714/c7089837/attachment.html>


More information about the llvm-commits mailing list