From 6ce5fbda5c3efc76502c212c01d5919f93acbd33 Mon Sep 17 00:00:00 2001 From: Dmitri Gribenko Date: Wed, 14 Aug 2019 11:35:04 +0000 Subject: [PATCH] Removed ToolExecutor::isSingleProcess, it is not used by anything Subscribers: cfe-commits Tags: #clang Differential Revision: https://reviews.llvm.org/D66212 git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@368832 91177308-0d34-0410-b5e6-96231b3b80d8 --- include/clang/Tooling/AllTUsExecution.h | 2 -- include/clang/Tooling/Execution.h | 7 ------- include/clang/Tooling/StandaloneExecution.h | 2 -- unittests/Tooling/ExecutionTest.cpp | 2 -- 4 files changed, 13 deletions(-) diff --git a/include/clang/Tooling/AllTUsExecution.h b/include/clang/Tooling/AllTUsExecution.h index f051119392..1e618b5ba2 100644 --- a/include/clang/Tooling/AllTUsExecution.h +++ b/include/clang/Tooling/AllTUsExecution.h @@ -44,8 +44,6 @@ public: StringRef getExecutorName() const override { return ExecutorName; } - bool isSingleProcess() const override { return true; } - using ToolExecutor::execute; llvm::Error diff --git a/include/clang/Tooling/Execution.h b/include/clang/Tooling/Execution.h index 74f0df5a5b..ca6f22c5da 100644 --- a/include/clang/Tooling/Execution.h +++ b/include/clang/Tooling/Execution.h @@ -115,13 +115,6 @@ public: /// Returns the name of a specific executor. virtual StringRef getExecutorName() const = 0; - /// Should return true iff executor runs all actions in a single process. - /// Clients can use this signal to find out if they can collect results - /// in-memory (e.g. to avoid serialization costs of using ToolResults). - /// The single-process executors can still run multiple threads, but all - /// executions are guaranteed to share the same memory. - virtual bool isSingleProcess() const = 0; - /// Executes each action with a corresponding arguments adjuster. virtual llvm::Error execute(llvm::ArrayRef< diff --git a/include/clang/Tooling/StandaloneExecution.h b/include/clang/Tooling/StandaloneExecution.h index 5fbc1e479c..8db6229acf 100644 --- a/include/clang/Tooling/StandaloneExecution.h +++ b/include/clang/Tooling/StandaloneExecution.h @@ -52,8 +52,6 @@ public: StringRef getExecutorName() const override { return ExecutorName; } - bool isSingleProcess() const override { return true; } - using ToolExecutor::execute; llvm::Error diff --git a/unittests/Tooling/ExecutionTest.cpp b/unittests/Tooling/ExecutionTest.cpp index 31d5fe5c42..6c2c91c058 100644 --- a/unittests/Tooling/ExecutionTest.cpp +++ b/unittests/Tooling/ExecutionTest.cpp @@ -95,8 +95,6 @@ public: StringRef getExecutorName() const override { return ExecutorName; } - bool isSingleProcess() const override { return true; } - llvm::Error execute(llvm::ArrayRef, ArgumentsAdjuster>>) override { -- 2.50.1