diff options
author | Simon Pilgrim <llvm-dev@redking.me.uk> | 2017-01-12 11:13:51 +0000 |
---|---|---|
committer | Simon Pilgrim <llvm-dev@redking.me.uk> | 2017-01-12 11:13:51 +0000 |
commit | e821446fbcbb6a9832029bb70a9513e7e7629cdd (patch) | |
tree | 28f627776fa2df8357e302cb6eddfca9b6ca2093 /tools/llvm-xray | |
parent | 182166ec7e67a029448161090acc007633360f70 (diff) |
Fix windows buildbots building llvm-xray
2 issues:
1 - replaced unix-style pid_t with cross-platform llvm::sys::ProcessInfo::ProcessId
2 - fixed shadow variable warning in lambda expression
Reviewed by @filcab
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@291760 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'tools/llvm-xray')
-rw-r--r-- | tools/llvm-xray/xray-account.cc | 10 | ||||
-rw-r--r-- | tools/llvm-xray/xray-account.h | 11 |
2 files changed, 13 insertions, 8 deletions
diff --git a/tools/llvm-xray/xray-account.cc b/tools/llvm-xray/xray-account.cc index 7c2b88cb790..747b4d18000 100644 --- a/tools/llvm-xray/xray-account.cc +++ b/tools/llvm-xray/xray-account.cc @@ -428,11 +428,11 @@ static CommandRegistration Unused(&Account, []() -> Error { xray::InstrumentationMapExtractor Extractor(AccountInstrMap, InstrMapFormat, Err); if (auto E = handleErrors( - std::move(Err), [&](std::unique_ptr<StringError> E) -> Error { - if (E->convertToErrorCode() == std::errc::no_such_file_or_directory) - return Error::success(); - return Error(std::move(E)); - })) + std::move(Err), [&](std::unique_ptr<StringError> SE) -> Error { + if (SE->convertToErrorCode() == std::errc::no_such_file_or_directory) + return Error::success(); + return Error(std::move(SE)); + })) return E; raw_fd_ostream OS(AccountOutput, EC, sys::fs::OpenFlags::F_Text); diff --git a/tools/llvm-xray/xray-account.h b/tools/llvm-xray/xray-account.h index 763ea011de0..cc9ba897e53 100644 --- a/tools/llvm-xray/xray-account.h +++ b/tools/llvm-xray/xray-account.h @@ -19,6 +19,7 @@ #include <vector> #include "func-id-helper.h" +#include "llvm/Support/Program.h" #include "llvm/Support/raw_ostream.h" #include "llvm/XRay/XRayRecord.h" @@ -28,10 +29,13 @@ namespace xray { class LatencyAccountant { public: typedef std::map<int32_t, std::vector<uint64_t>> FunctionLatencyMap; - typedef std::map<pid_t, std::pair<uint64_t, uint64_t>> PerThreadMinMaxTSCMap; + typedef std::map<llvm::sys::ProcessInfo::ProcessId, + std::pair<uint64_t, uint64_t>> + PerThreadMinMaxTSCMap; typedef std::map<uint8_t, std::pair<uint64_t, uint64_t>> PerCPUMinMaxTSCMap; typedef std::vector<std::pair<int32_t, uint64_t>> FunctionStack; - typedef std::map<pid_t, FunctionStack> PerThreadFunctionStackMap; + typedef std::map<llvm::sys::ProcessInfo::ProcessId, FunctionStack> + PerThreadFunctionStackMap; private: PerThreadFunctionStackMap PerThreadFunctionStack; @@ -75,7 +79,8 @@ public: /// bool accountRecord(const XRayRecord &Record); - const FunctionStack *getThreadFunctionStack(pid_t TId) const { + const FunctionStack * + getThreadFunctionStack(llvm::sys::ProcessInfo::ProcessId TId) const { auto I = PerThreadFunctionStack.find(TId); if (I == PerThreadFunctionStack.end()) return nullptr; |