[Lldb-commits] [PATCH] D26170: Find clang resource directory via *nix-style lookup

2016-11-02 Thread Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL285838: Find clang resource directory via *nix-style lookup (authored by cbieneman). Changed prior to commit: https://reviews.llvm.org/D26170?vs=76496&id=76735#toc Repository: rL LLVM https://review

[Lldb-commits] [PATCH] D26170: Find clang resource directory via *nix-style lookup

2016-11-01 Thread Pavel Labath via lldb-commits
labath added a comment. looks good in general, but please check whether this is correct with LLVM_LIBDIR_SUFFIX=64. Comment at: source/Host/posix/HostInfoPosix.cpp:176 + file_spec, + llvm::Twine("/lib/clang/", llvm::StringRef(CLANG_VERSION_STRING)).str()); +} ---

[Lldb-commits] [PATCH] D26170: Find clang resource directory via *nix-style lookup

2016-10-31 Thread Todd Fiala via lldb-commits
tfiala accepted this revision. tfiala added a comment. This revision is now accepted and ready to land. Looks reasonable. Comment at: source/Host/macosx/HostInfoMacOSX.mm:235-239 size_t framework_pos = raw_path.find("LLDB.framework"); - if (framework_pos != std::string::npo

[Lldb-commits] [PATCH] D26170: Find clang resource directory via *nix-style lookup

2016-10-31 Thread Chris Bieneman via lldb-commits
beanz created this revision. beanz added reviewers: zturner, labath, spyffe, tfiala. beanz added a subscriber: lldb-commits. This patch allows the Darwin build to fall back to to Posix-style lookups for the clang resource directory if the debugger library isn't inside a framework. The patch also