Skip to content

Commit 1517acf

Browse files
samloeschergeorgepaw
authored andcommitted
Remove multiReadST from gateway target options
Summary: Removed option from poplar, the better solution is T43412, but for now this fixes the build Ref T43366 Test Plan: Builds on CI Reviewers: georgep, #tensorflow, #framework_ip_review_-_any_oss_or_third-party_code_use_has_been_approved, jakeh Reviewed By: #tensorflow, #framework_ip_review_-_any_oss_or_third-party_code_use_has_been_approved, jakeh Maniphest Tasks: T43366 Differential Revision: https://phabricator.sourcevertex.net/D52617
1 parent ff3c30a commit 1517acf

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

tensorflow/compiler/plugin/poplar/driver/poplar_executor.cc

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2044,7 +2044,6 @@ Status PoplarExecutor::ConfigurePoplarDevice(const IpuOptions& cfg) {
20442044
target_hash.push_back(ipu_.Target().getIpuLinkDomainSize());
20452045
target_hash.push_back(static_cast<int64>(ipu_.Target().getIpuLinkTopology()));
20462046
target_hash.push_back(ipu_.Target().getGatewayMode());
2047-
target_hash.push_back(ipu_.Target().getGatewayMultiReadServiceTable());
20482047

20492048
if (ipu_.Target().getTargetType() == poplar::TargetType::IPU) {
20502049
target_hash.push_back(

0 commit comments

Comments
 (0)