Skip to content

Commit c4a2f46

Browse files
hakosgeorgepaw
authored andcommitted
Add gateway mode options to target hash
Summary: We need to recompile if any of these change as they affect code generation. Hopefully we will get a way to hash all the target options from Poplar as a part of T43412 so we do not have to keep maintaining this here. Fixes T43452. Reviewers: #tensorflow, simonl, #framework_ip_review_-_any_oss_or_third-party_code_use_has_been_approved, vladimirm Reviewed By: #tensorflow, #framework_ip_review_-_any_oss_or_third-party_code_use_has_been_approved, vladimirm Maniphest Tasks: T43452 Differential Revision: https://phabricator.sourcevertex.net/D49112
1 parent a1c6041 commit c4a2f46

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

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

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2042,6 +2042,9 @@ Status PoplarExecutor::ConfigurePoplarDevice(const IpuOptions& cfg) {
20422042
static_cast<int64>(ipu_.Target().getIpuLinkConfiguration()));
20432043
target_hash.push_back(ipu_.Target().getIpuLinkDomainSize());
20442044
target_hash.push_back(static_cast<int64>(ipu_.Target().getIpuLinkTopology()));
2045+
target_hash.push_back(ipu_.Target().getGatewayMode());
2046+
target_hash.push_back(ipu_.Target().getGatewayMultiReadServiceTable());
2047+
20452048
if (ipu_.Target().getTargetType() == poplar::TargetType::IPU) {
20462049
target_hash.push_back(
20472050
std::hash<string>()(ipu_.Target().getTargetArchString()));

0 commit comments

Comments
 (0)