[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