gribozavr created this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
gribozavr added a reviewer: sammccall.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D66212
Files:
clang/include/clang/Tooling/AllTUsExecution.h
clang/include/clang/Tooling/Execution.h
clang/include/clang/Tooling/StandaloneExecution.h
clang/unittests/Tooling/ExecutionTest.cpp
Index: clang/unittests/Tooling/ExecutionTest.cpp
===================================================================
--- clang/unittests/Tooling/ExecutionTest.cpp
+++ clang/unittests/Tooling/ExecutionTest.cpp
@@ -95,8 +95,6 @@
StringRef getExecutorName() const override { return ExecutorName; }
- bool isSingleProcess() const override { return true; }
-
llvm::Error
execute(llvm::ArrayRef<std::pair<std::unique_ptr<FrontendActionFactory>,
ArgumentsAdjuster>>) override {
Index: clang/include/clang/Tooling/StandaloneExecution.h
===================================================================
--- clang/include/clang/Tooling/StandaloneExecution.h
+++ clang/include/clang/Tooling/StandaloneExecution.h
@@ -52,8 +52,6 @@
StringRef getExecutorName() const override { return ExecutorName; }
- bool isSingleProcess() const override { return true; }
-
using ToolExecutor::execute;
llvm::Error
Index: clang/include/clang/Tooling/Execution.h
===================================================================
--- clang/include/clang/Tooling/Execution.h
+++ clang/include/clang/Tooling/Execution.h
@@ -115,13 +115,6 @@
/// 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<
Index: clang/include/clang/Tooling/AllTUsExecution.h
===================================================================
--- clang/include/clang/Tooling/AllTUsExecution.h
+++ clang/include/clang/Tooling/AllTUsExecution.h
@@ -44,8 +44,6 @@
StringRef getExecutorName() const override { return ExecutorName; }
- bool isSingleProcess() const override { return true; }
-
using ToolExecutor::execute;
llvm::Error
Index: clang/unittests/Tooling/ExecutionTest.cpp
===================================================================
--- clang/unittests/Tooling/ExecutionTest.cpp
+++ clang/unittests/Tooling/ExecutionTest.cpp
@@ -95,8 +95,6 @@
StringRef getExecutorName() const override { return ExecutorName; }
- bool isSingleProcess() const override { return true; }
-
llvm::Error
execute(llvm::ArrayRef<std::pair<std::unique_ptr<FrontendActionFactory>,
ArgumentsAdjuster>>) override {
Index: clang/include/clang/Tooling/StandaloneExecution.h
===================================================================
--- clang/include/clang/Tooling/StandaloneExecution.h
+++ clang/include/clang/Tooling/StandaloneExecution.h
@@ -52,8 +52,6 @@
StringRef getExecutorName() const override { return ExecutorName; }
- bool isSingleProcess() const override { return true; }
-
using ToolExecutor::execute;
llvm::Error
Index: clang/include/clang/Tooling/Execution.h
===================================================================
--- clang/include/clang/Tooling/Execution.h
+++ clang/include/clang/Tooling/Execution.h
@@ -115,13 +115,6 @@
/// 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<
Index: clang/include/clang/Tooling/AllTUsExecution.h
===================================================================
--- clang/include/clang/Tooling/AllTUsExecution.h
+++ clang/include/clang/Tooling/AllTUsExecution.h
@@ -44,8 +44,6 @@
StringRef getExecutorName() const override { return ExecutorName; }
- bool isSingleProcess() const override { return true; }
-
using ToolExecutor::execute;
llvm::Error
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
