llvm-config: Don't check for "lib" prefix

[jvesely]

Summary

GetComponentLibraryNameSlice() already checks that.

Using BUILD_SHARED_LIBS
AC->Library provides library name without prefix or suffix so this check would always fail.

Fixes libtool linking breakage in mesa since r206263: llvm-config: Add preliminary Windows support
--libs output should be different from --libfiles, the latter cannot be used with libtool

[• ehsan]

I think http://reviews.llvm.org/D17300 is a better solution for this. Can you please try that patch?

[jvesely]

In D17358#358997, @ehsan wrote:

I think http://reviews.llvm.org/D17300 is a better solution for this. Can you please try that patch?

D17300 works for me as well. abandoning this one.
thank you.

tools/llvm-config/llvm-config.cpp

if (PrintLibs || PrintLibNames || PrintLibFiles) {
         const bool Shared = LinkMode == LinkModeShared;
         if (PrintLibNames) {
           OS << GetComponentLibraryFileName(Lib, Shared);
         } else if (PrintLibFiles) {
           OS << GetComponentLibraryPath(Lib, Shared);
         } else if (PrintLibs) {
           // If this is a typical library name, include it using -l.
           StringRef LibName;
-          if (Lib.startswith("lib")) {
-            if (GetComponentLibraryNameSlice(Lib, LibName)) {
-              OS << "-l" << LibName;
-            } else {
-              OS << "-l:" << GetComponentLibraryFileName(Lib, Shared);
-            }
-          } else {
-            // Otherwise, print the full path.
-            OS << GetComponentLibraryPath(Lib, Shared);
-          }
         }
       };
 
       if (LinkMode == LinkModeShared && !BuiltSharedLibs) {
         PrintForLib(DyLibName);
       } else {
         for (unsigned i = 0, e = RequiredLibs.size(); i != e; ++i) {
           auto Lib = RequiredLibs[i];