[PATCH] D35739: Fix LLVMgold plugin name/path for non-Linux.

2017-08-14 Thread Dan Albert via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL310895: Fix LLVMgold plugin name/path for non-Linux. (authored by danalbert). Repository: rL LLVM https://reviews.llvm.org/D35739 Files: cfe/trunk/lib/Driver/ToolChains/CommonArgs.cpp Index: cfe/t

[PATCH] D35739: Fix LLVMgold plugin name/path for non-Linux.

2017-07-21 Thread Dan Albert via Phabricator via cfe-commits
danalbert added a comment. In https://reviews.llvm.org/D35739#817705, @srhines wrote: > It's hard to believe that there is nothing in Support that gives you the > proper suffix for shared libraries. It would seem like this might be useful > elsewhere, but I actually didn't find really any other

[PATCH] D35739: Fix LLVMgold plugin name/path for non-Linux.

2017-07-21 Thread Stephen Hines via Phabricator via cfe-commits
srhines added a comment. Looks great. Thanks for fixing this up. It's hard to believe that there is nothing in Support that gives you the proper suffix for shared libraries. It would seem like this might be useful elsewhere, but I actually didn't find really any other place that wants to use th

[PATCH] D35739: Fix LLVMgold plugin name/path for non-Linux.

2017-07-21 Thread Dan Albert via Phabricator via cfe-commits
danalbert created this revision. It's only named LLVMgold.so on Linux. Fix the name for Windows and Darwin. Also fix the path for Windows so binutils doesn't have to. Repository: rL LLVM https://reviews.llvm.org/D35739 Files: lib/Driver/ToolChains/CommonArgs.cpp Index: lib/Drive