aboutsummaryrefslogtreecommitdiffhomepage
path: root/tensorflow/compiler/tf2xla
diff options
context:
space:
mode:
authorGravatar Yunxing Dai <yunxing@google.com>2018-09-13 20:36:01 -0700
committerGravatar TensorFlower Gardener <gardener@tensorflow.org>2018-09-13 20:39:26 -0700
commit40010e6287980f63158807aef163276ed1cce272 (patch)
tree29a1f590e732f07cdb961ae037d52707a1b734c6 /tensorflow/compiler/tf2xla
parent4a665550dacdb5e162e71b4afe039de178ffc49f (diff)
[GraphCompiler] Remove the use of XLA context as arugmnet.
- XLAContext is never used in the class, remove it from member list. - Be more clear in the comment that the result is written to the context from the compilation device. PiperOrigin-RevId: 212924213
Diffstat (limited to 'tensorflow/compiler/tf2xla')
-rw-r--r--tensorflow/compiler/tf2xla/graph_compiler.h13
-rw-r--r--tensorflow/compiler/tf2xla/xla_compiler.cc3
2 files changed, 7 insertions, 9 deletions
diff --git a/tensorflow/compiler/tf2xla/graph_compiler.h b/tensorflow/compiler/tf2xla/graph_compiler.h
index ab7cac7100..e9f02201cf 100644
--- a/tensorflow/compiler/tf2xla/graph_compiler.h
+++ b/tensorflow/compiler/tf2xla/graph_compiler.h
@@ -55,17 +55,17 @@ namespace tensorflow {
// op registration infrastructure instead of FunctionLibraryRuntime.
class GraphCompiler {
public:
- GraphCompiler(XlaContext* xla_context, XlaCompilationDevice* device,
- Graph* graph, FunctionLibraryRuntime* flib,
+ GraphCompiler(XlaCompilationDevice* device, Graph* graph,
+ FunctionLibraryRuntime* flib,
ScopedStepContainer* step_container)
- : xla_context_(xla_context),
- device_(device),
+ : device_(device),
graph_(graph),
flib_(flib),
step_container_(step_container) {}
- // Compiles the graph. The results are written in `xla_context` that is passed
- // into the compiler.
+ // Compiles the graph. The results are written in xla_context stored in the
+ // resource_manager of the 'XlaCompilationDevice' that's passed into the
+ // constructor.
Status Compile();
private:
@@ -82,7 +82,6 @@ class GraphCompiler {
// using `compiler_`.
Status CompileFunctionalNode(Node* n, OpKernelContext* op_context);
- XlaContext* xla_context_;
XlaCompilationDevice* device_;
Graph* graph_;
FunctionLibraryRuntime* flib_;
diff --git a/tensorflow/compiler/tf2xla/xla_compiler.cc b/tensorflow/compiler/tf2xla/xla_compiler.cc
index 105f3b61d5..739e47778a 100644
--- a/tensorflow/compiler/tf2xla/xla_compiler.cc
+++ b/tensorflow/compiler/tf2xla/xla_compiler.cc
@@ -325,8 +325,7 @@ Status ExecuteGraph(XlaContext* xla_context, std::unique_ptr<Graph> graph,
step_container->name(), XlaContext::kXlaContextResourceName,
xla_context));
- GraphCompiler graph_compiler(xla_context, device, graph.get(), flib,
- step_container.get());
+ GraphCompiler graph_compiler(device, graph.get(), flib, step_container.get());
TF_RETURN_IF_ERROR(graph_compiler.Compile());
// Explicitly clean up the step container, to capture the cleanup status.
step_container.reset();