This is an archive of the discontinued LLVM Phabricator instance.

Added a separate install target for compilert-rt-headers
AbandonedPublic

Authored by krytarowski on Dec 13 2017, 2:41 AM.

Details

Reviewers
alekseyshl
beanz
Summary

This patch adds a new install target install-compilert-rt-headers,
that is similar to install-clang-headers.

It allows to install the headers without installing all of
compiler-rt.

Event Timeline

ilya-biryukov created this revision.Dec 13 2017, 2:41 AM
Herald added subscribers: Restricted Project, mgorny. · View Herald TranscriptDec 13 2017, 2:41 AM
krytarowski commandeered this revision.Dec 13 2017, 5:27 AM
krytarowski edited reviewers, added: ilya-biryukov; removed: krytarowski.

Hmm, I attempted to give up from reviewing, not taking over. I have no opinion on this change.

@krytarowski, maybe you could point me to someone who could review this?

ilya-biryukov edited reviewers, added: beanz; removed: ilya-biryukov.Dec 13 2017, 7:54 AM
krytarowski abandoned this revision.Dec 13 2017, 8:45 AM

Looking at CODE_OWNERS, maybe:

N: Alexey Samsonov
E: samsonov@google.com
D: CMake build, test suite

Please take this patch (commander option).

kcc added a subscriber: kcc.Dec 13 2017, 9:45 AM

Aleksey, please review.
Please also remove samsonov@ from owners (he is not active in LLVM any more, AFAICT) and replace with yourself.

I don't think I can "command" the patch back to me. (It's not in the list of action, at least). I'll resend it instead.