diff options
| author | Craig Topper <craig.topper@intel.com> | 2017-07-18 02:41:12 +0000 |
|---|---|---|
| committer | Craig Topper <craig.topper@intel.com> | 2017-07-18 02:41:12 +0000 |
| commit | 9e465894f8127ea058fa35e3744e1ec83fe24b48 (patch) | |
| tree | d23d003cec0c32acbba4257632ae70ff384cd35f /clang/lib/FrontendTool/ExecuteCompilerInvocation.cpp | |
| parent | 5d77c20a7a06b3390c5e41e9c2bbc4f66edeee7d (diff) | |
| download | llvm-9e465894f8127ea058fa35e3744e1ec83fe24b48.zip llvm-9e465894f8127ea058fa35e3744e1ec83fe24b48.tar.gz llvm-9e465894f8127ea058fa35e3744e1ec83fe24b48.tar.bz2 | |
[Analysis] RemoveTotalMemInst counting in InstCount to avoid reading back other Statistic variables
Summary:
Previously, we counted TotalMemInst by reading certain instruction counters before and after calling visit and then finding the difference. But that wouldn't be thread safe if this same pass was being ran on multiple threads.
This list of "memory instructions" doesn't make sense to me as it includes call/invoke and is missing atomics.
This patch removes the counter all together.
Reviewers: hfinkel, chandlerc, davide
Reviewed By: davide
Subscribers: davide, llvm-commits
Differential Revision: https://reviews.llvm.org/D33608
llvm-svn: 308260
Diffstat (limited to 'clang/lib/FrontendTool/ExecuteCompilerInvocation.cpp')
0 files changed, 0 insertions, 0 deletions