Skip to content

Commit

Permalink
Remove dead code. NFC
Browse files Browse the repository at this point in the history
  • Loading branch information
vgvassilev committed Dec 30, 2024
1 parent a2e1c50 commit cc8d68a
Showing 1 changed file with 4 additions and 14 deletions.
18 changes: 4 additions & 14 deletions lib/Differentiator/ReverseModeVisitor.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -192,7 +192,6 @@ Expr* getArraySizeExpr(const ArrayType* AT, ASTContext& context,
}

DerivativeAndOverload ReverseModeVisitor::Derive() {
const FunctionDecl* FD = m_DiffReq.Function;
if (m_ExternalSource)
m_ExternalSource->ActOnStartOfDerive();
if (m_DiffReq.Mode == DiffMode::error_estimation)
Expand All @@ -204,13 +203,8 @@ Expr* getArraySizeExpr(const ArrayType* AT, ASTContext& context,
assert(m_DiffReq.Function && "Must not be null.");

DiffParams args{};
if (m_DiffReq.Args)
for (const auto& dParam : m_DiffReq.DVI)
args.push_back(dParam.param);
else
std::copy(FD->param_begin(), FD->param_end(), std::back_inserter(args));
if (args.empty())
return {};
for (const auto& dParam : m_DiffReq.DVI)
args.push_back(dParam.param);

if (m_ExternalSource)
m_ExternalSource->ActAfterParsingDiffArgs(m_DiffReq, args);
Expand Down Expand Up @@ -350,7 +344,6 @@ Expr* getArraySizeExpr(const ArrayType* AT, ASTContext& context,
}

DerivativeAndOverload ReverseModeVisitor::DerivePullback() {
const clang::FunctionDecl* FD = m_DiffReq.Function;
// FIXME: Duplication of external source here is a workaround
// for the two 'Derive's being different functions.
if (m_ExternalSource)
Expand All @@ -359,11 +352,8 @@ Expr* getArraySizeExpr(const ArrayType* AT, ASTContext& context,
assert(m_DiffReq.Function && "Must not be null.");

DiffParams args{};
if (!m_DiffReq.DVI.empty())
for (const auto& dParam : m_DiffReq.DVI)
args.push_back(dParam.param);
else
std::copy(FD->param_begin(), FD->param_end(), std::back_inserter(args));
for (const auto& dParam : m_DiffReq.DVI)
args.push_back(dParam.param);
#ifndef NDEBUG
bool isStaticMethod = utils::IsStaticMethod(FD);
assert((!args.empty() || !isStaticMethod) &&
Expand Down

0 comments on commit cc8d68a

Please sign in to comment.