diff options
author | 2017-05-18 13:13:38 -0700 | |
---|---|---|
committer | 2017-05-18 13:17:14 -0700 | |
commit | e84588a3e9e67d91d6ae32e64469f890d217c5dd (patch) | |
tree | fab8672a31d1e2e7477aa1ff197bb3098fe516f5 /tensorflow/compiler/xla/service/hlo_cost_analysis_test.cc | |
parent | 315f195f728d59647f23f10ecc664b8ce9466a06 (diff) |
[XLA] Attach an HloModuleConfig to HloModule, obviating the need to pass them around as a pair.
This cuts through a bunch of critical XLA APIs, but it's time... The background for this change is to make flags/options more easily pipe-able from the TF/XLA boundary deep into the XLA compiler and other components.
The situation after this CL is still not perfect; there are a number of places with chicken-egg scenarios when a module has to be constructed before a config (to register the result shape), but the situation is strictly better than before. Future CLs will clean things up even more.
PiperOrigin-RevId: 156469639
Diffstat (limited to 'tensorflow/compiler/xla/service/hlo_cost_analysis_test.cc')
-rw-r--r-- | tensorflow/compiler/xla/service/hlo_cost_analysis_test.cc | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/tensorflow/compiler/xla/service/hlo_cost_analysis_test.cc b/tensorflow/compiler/xla/service/hlo_cost_analysis_test.cc index 22e782da27..f71ffeb887 100644 --- a/tensorflow/compiler/xla/service/hlo_cost_analysis_test.cc +++ b/tensorflow/compiler/xla/service/hlo_cost_analysis_test.cc @@ -126,8 +126,10 @@ class HloCostAnalysisTest : public ::testing::Test { auto user_computation = user_computation_status.ConsumeValueOrDie(); VersionedComputationHandle versioned_handle = user_computation->GetVersionedHandle(); - return std::move( - computation_tracker_.BuildHloModule(versioned_handle).ValueOrDie()); + return std::move(computation_tracker_ + .BuildHloModule(versioned_handle, + /*config=*/nullptr) + .ValueOrDie()); } Client* client_; |