echristo added a subscriber: echristo.
echristo added a comment.
Oh, sure I can.
Done thusly:
dzur:~/sources/llvm/tools/clang> git svn dcommit
Committing to https://llvm.org/svn/llvm-project/cfe/trunk ...
M include/clang/Driver/ToolChain.h
M lib/Driver/ToolChain.cpp
M tools/driver/driver.cpp
Com
Oh, sure I can.
Done thusly:
dzur:~/sources/llvm/tools/clang> git svn dcommit
Committing to https://llvm.org/svn/llvm-project/cfe/trunk ...
M include/clang/Driver/ToolChain.h
M lib/Driver/ToolChain.cpp
M tools/driver/driver.cpp
Committed r248592
-eric
On Fri, Sep 25, 2015 at 10:20 AM Luke Zark
zarko added a comment.
Great!
I don't have commit access; would someone who does please land this?
Repository:
rL LLVM
http://reviews.llvm.org/D12982
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/l
echristo added a comment.
Nothing comes to mind.
Thanks!
-eric
Repository:
rL LLVM
http://reviews.llvm.org/D12982
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
zarko added a comment.
This should be a straight move, unless there is something I don't understand
about the semantics of `llvm::InitializeAllTargets();` that would make moving
it earlier in `main` a bad idea.
Repository:
rL LLVM
http://reviews.llvm.org/D12982
__
echristo accepted this revision.
echristo added a comment.
This revision is now accepted and ready to land.
Looks like a straight move? If so, LGTM, if not can you explain what changed?
Repository:
rL LLVM
http://reviews.llvm.org/D12982
___
cfe-c
zarko created this revision.
zarko added reviewers: rnk, echristo.
zarko added a subscriber: cfe-commits.
zarko set the repository for this revision to rL LLVM.
Herald added subscribers: srhines, danalbert, tberghammer.
The Clang gcc-compatible driver (clang/tools/driver/driver.cpp) has some logic
