[PATCH][LLD] CODE_OWNERS.TXT

Rui Ueyama ruiu at google.com
Fri Jul 24 17:15:20 PDT 2015


LGTM with this fix.

> +D: COFF (COFF/*), ELF backends

ELF backend has just been submitted, so this can be "COFF, ELF backends
(COFF/* ELF/*)"

On Fri, Jul 24, 2015 at 4:56 PM, Nick Kledzik <kledzik at apple.com> wrote:

>
> On Jul 23, 2015, at 5:33 PM, Rui Ueyama <ruiu at google.com> wrote:
>
> On Thu, Jul 23, 2015 at 5:02 PM, Nick Kledzik <kledzik at apple.com> wrote:
>
>>
>> On Jul 23, 2015, at 4:36 PM, Rui Ueyama <ruiu at google.com> wrote:
>>
>> > +N: Rui Ueyama
>> > +E: ruiu at google.com
>> > +D: COFF, ELF backends
>>
>> You may want to also mention about directories like we do in
>> clang/CODE_OWNERS.TXT.
>>
>> COFF, ELF backends (COFF/* and ELF/*)
>>
>> Except ELF/* does not exist yet.
>>
>
> Correct. So it's only for COFF/* for now.
>
>
> New patch attached.
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150724/8ae89230/attachment.html>


More information about the llvm-commits mailing list