Skip to content

Commit

Permalink
Remove FIN_OLD_PROBLEM_DESCRIPTION_COMPAT (#2503)
Browse files Browse the repository at this point in the history
  • Loading branch information
averinevg authored Dec 19, 2023
1 parent dd23f4e commit 10250d6
Show file tree
Hide file tree
Showing 4 changed files with 3 additions and 32 deletions.
2 changes: 1 addition & 1 deletion fin
Submodule fin updated from ae2ff1 to 044f5e
23 changes: 0 additions & 23 deletions src/include/miopen/problem_description.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -33,8 +33,6 @@
#include <cstdint>
#include <string>

#define FIN_OLD_PROBLEM_DESCRIPTION_COMPAT 1

namespace miopen {

// Tensor Helper APIs
Expand All @@ -57,27 +55,6 @@ SetDescFromMLDesc(int spatial_dims, TTo& to, const TensorDescriptor& tensor, con
return tensor.GetElementSpace();
}

#if FIN_OLD_PROBLEM_DESCRIPTION_COMPAT
struct ProblemDescription : conv::ProblemDescription
{
ProblemDescription() = default;

ProblemDescription(conv::ProblemDescription desc) : conv::ProblemDescription(std::move(desc))
{
conv_problem.p = this;
}

struct
{
void SetupFloats(ExecutionContext& ctx) const { p->SetupFloats(ctx); }

private:
const conv::ProblemDescription* p = nullptr;
friend struct ProblemDescription;
} conv_problem;
};
#endif

// For mlo_construct_base
// TODO remove this
struct ProblemDescriptionCompatTemporary
Expand Down
5 changes: 1 addition & 4 deletions src/solver/norm/forward_layernorm2d_ck.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -160,10 +160,7 @@ typename LnPtrsType::iterator FindLnPtr(LnPtrsType& ln_ptrs,
});
}

template <typename DeviceOpType,
typename CKArgsType,
typename CastType,
typename ProblemDescriptionType = ProblemDescription>
template <typename DeviceOpType, typename CKArgsType, typename CastType>
ConvSolution MakeInvokerFactory([[maybe_unused]] const ExecutionContext& context,
const miopen::norm::ProblemDescription& problem)
{
Expand Down
5 changes: 1 addition & 4 deletions src/solver/norm/forward_layernorm4d_ck.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -168,10 +168,7 @@ typename LnPtrsType::iterator FindLnPtr(LnPtrsType& ln_ptrs,
});
}

template <typename DeviceOpType,
typename CKArgsType,
typename CastType,
typename ProblemDescriptionType = ProblemDescription>
template <typename DeviceOpType, typename CKArgsType, typename CastType>
ConvSolution MakeInvokerFactory([[maybe_unused]] const ExecutionContext& context,
const miopen::norm::ProblemDescription& problem)
{
Expand Down

0 comments on commit 10250d6

Please sign in to comment.