mgorny added a comment.
Herald added a subscriber: llvm-commits.
I'm sorry for being late to the party but could you update
`unittests/Driver/DistroTest.cpp` as well?
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D39588/new/
https://reviews.llvm.org/D39588
This revision was automatically updated to reflect the committed changes.
Closed by commit rL318608: [Driver] add initial support for alpine linux
(authored by martell).
Changed prior to commit:
https://reviews.llvm.org/D39588?vs=121462&id=123488#toc
Repository:
rL LLVM
https://reviews.llvm
rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.
lgtm
Repository:
rL LLVM
https://reviews.llvm.org/D39588
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailm
martell added a comment.
@rnk ping.
Feel free to add anyone you think might be more suitable if you can't review
this.
Repository:
rL LLVM
https://reviews.llvm.org/D39588
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm
martell updated this revision to Diff 121462.
martell edited the summary of this revision.
Repository:
rL LLVM
https://reviews.llvm.org/D39588
Files:
include/clang/Driver/Distro.h
lib/Driver/Distro.cpp
lib/Driver/ToolChains/Linux.cpp
test/Driver/pic.c
Index: test/Driver/pic.c
===
martell created this revision.
Add alpine linux as a distro.
We support building for musl since https://reviews.llvm.org/rL292848.
Let's add this important docker / kubernetes empowering distro.
Some bits are still missing but this is a good base.
Repository:
rL LLVM
https://reviews.llvm.org