Symbols::LocateExecutableSymbolFile() shouldn't try to look for files in /usr/lib/debug on Windows

[enlight]

Summary

/usr/lib/debug doesn't exist on Windows so there's no point even attempting to look for symbol files in there.

[enlight]

Zach,

I've looked at the path separator issue you've mentioned earlier in D13201, and I think it's fine as is. All those paths with the forward slashes end up in a FileSpec, which converts forward slashes to backward slashes before passing them through to any Win32 APIs, and FileSpec::GetPath() also converts slashes by default.

[zturner]

Ok, sounds good then. Thanks for checking

lldb/trunk/source/Host/common/Symbols.cpp

if (symbol_filename && symbol_filename[0])
 
         // Add module directory.
         const ConstString &file_dir = module_spec.GetFileSpec().GetDirectory();
         debug_file_search_paths.AppendIfUnique (FileSpec(file_dir.AsCString("."), true));
 
         // Add current working directory.
         debug_file_search_paths.AppendIfUnique (FileSpec(".", true));
 
+#ifndef LLVM_ON_WIN32
         // Add /usr/lib/debug directory.
         debug_file_search_paths.AppendIfUnique (FileSpec("/usr/lib/debug", true));
+#endif // LLVM_ON_WIN32
 
         std::string uuid_str;
         const UUID &module_uuid = module_spec.GetUUID();
         if (module_uuid.IsValid())
         {
             // Some debug files are stored in the .build-id directory like this:
             //   /usr/lib/debug/.build-id/ff/e7fe727889ad82bb153de2ad065b2189693315.debug
             uuid_str = module_uuid.GetAsString("");