Skip to content

Commit

Permalink
Compute and process Differentiation Request graph
Browse files Browse the repository at this point in the history
Plan for dynamic graph
- The relations between different differentiation requests can be modelled as a graph.
For example, if `f_a` calls `f_b`, there will be two differentiation requests `df_a` and `df_b`,
the edge between them can be understood as `created_because_of`. This also means that the functions
called by the users to be explicitly differentiated (or `DiffRequests` created because of these) are the source nodes,
i.e. no incoming edges. In most cases, this graph aligns with the call graph, but in some cases,
the graph depends on the internal implementation, like the Hessian computation, which requires creating multiple
`fwd_mode` requests followed by a `rev_mode` request.

- We can use this graph to order the computation of differentiation requests. This was already being done implicitly
in the initial recursive implementation. Whenever we encountered a call expression, we started differentiation of the
called function; this was sort of like a depth-first search strategy.
  - This had problems, as `Clang` reported errors when it encountered a new function scope (of the derivative of the called function) in the middle of the old function scope (of the derivative of the callee function). It treated the nested one like a lambda expression. The issue regarding this: #745.

- To fix this, an initial strategy was to eliminate the recursive approach. Hence, a queue-based breadth-first approach
was implemented in this PR: #848.
  - Although it fixed the problem, the graph traversal was still implicit. We needed some way to compute/store the complete graph and possibly optimize it, such as converting edges to model the `requires_derivative_of` relation. Using this, we could proceed with differentiation in a topologically sorted ordering.
  - It also required one caveat: although we don't differentiate the called function completely in a recursive way, we still need to declare it so that we can have the call expression completed (i.e. `auto t0 = f_pushforward(...)`).

- To move towards the final stage of having the complete graph computed before starting the differentiation, we need the complete information on how the `DiffRequest` will be formed inside the visitors (including arguments or `DVI` info).
This whole approach will require activity analysis in the first pass.
  - As an incremental improvement, the first requirement was to implement infrastructure to support explicit modelling of the graph and use that to have a breadth-first traversal (and eventually topological ordering).

This is the initial PR for capturing the differentiation plan in a graphical format.

However, the traversal order is still breadth-first, as we don't have the complete graph in the first pass - mainly because of a lack of information about the args required for `pushforward` and `pullbacks`.

This can be improved with the help of activity analysis to capture the complete graph in the first pass, processing the plan in a topologically sorted manner and pruning the graph for user-defined functions. I started this with this approach, and the initial experimental commit is available here for future reference: vaithak@82c0b42.
  • Loading branch information
vaithak authored and vgvassilev committed May 3, 2024
1 parent 922bd1c commit b59c9aa
Show file tree
Hide file tree
Showing 16 changed files with 398 additions and 67 deletions.
17 changes: 13 additions & 4 deletions include/clad/Differentiator/DerivativeBuilder.h
Original file line number Diff line number Diff line change
Expand Up @@ -38,9 +38,6 @@ namespace clad {
class CladPlugin;
clang::FunctionDecl* ProcessDiffRequest(CladPlugin& P,
DiffRequest& request);
// FIXME: This function should be removed and the entire plans array
// should be somehow made accessible to all the visitors.
void AddRequestToSchedule(CladPlugin& P, const DiffRequest& request);
} // namespace plugin

} // namespace clad
Expand Down Expand Up @@ -87,6 +84,7 @@ namespace clad {
plugin::CladPlugin& m_CladPlugin;
clang::ASTContext& m_Context;
const DerivedFnCollector& m_DFC;
clad::DynamicGraph<DiffRequest>& m_DiffRequestGraph;
std::unique_ptr<utils::StmtClone> m_NodeCloner;
clang::NamespaceDecl* m_BuiltinDerivativesNSD;
/// A reference to the model to use for error estimation (if any).
Expand Down Expand Up @@ -137,7 +135,8 @@ namespace clad {

public:
DerivativeBuilder(clang::Sema& S, plugin::CladPlugin& P,
const DerivedFnCollector& DFC);
const DerivedFnCollector& DFC,
clad::DynamicGraph<DiffRequest>& DRG);
~DerivativeBuilder();
/// Reset the model use for error estimation (if any).
/// \param[in] estModel The error estimation model, can be either
Expand Down Expand Up @@ -172,6 +171,16 @@ namespace clad {
///
/// \returns The derived function if found, nullptr otherwise.
clang::FunctionDecl* FindDerivedFunction(const DiffRequest& request);
/// Add edge from current request to the given request in the DiffRequest
/// graph.
///
/// \param[in] request The request to add the edge to.
void AddEdgeToGraph(const DiffRequest& request);
/// Add edge between two requests in the DiffRequest graph.
///
/// \param[in] from The source request.
/// \param[in] to The destination request.
void AddEdgeToGraph(const DiffRequest& from, const DiffRequest& to);
};

} // end namespace clad
Expand Down
36 changes: 36 additions & 0 deletions include/clad/Differentiator/DiffMode.h
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,42 @@ enum class DiffMode {
reverse_mode_forward_pass,
error_estimation
};

/// Convert enum value to string.
inline const char* DiffModeToString(DiffMode mode) {
switch (mode) {
case DiffMode::forward:
return "forward";
case DiffMode::vector_forward_mode:
return "vector_forward_mode";
case DiffMode::experimental_pushforward:
return "pushforward";
case DiffMode::experimental_pullback:
return "pullback";
case DiffMode::experimental_vector_pushforward:
return "vector_pushforward";
case DiffMode::reverse:
return "reverse";
case DiffMode::hessian:
return "hessian";
case DiffMode::jacobian:
return "jacobian";
case DiffMode::reverse_mode_forward_pass:
return "reverse_mode_forward_pass";
case DiffMode::error_estimation:
return "error_estimation";
default:
return "unknown";
}
}

/// Returns true if the given mode is a pullback/pushforward mode.
inline bool IsPullbackOrPushforwardMode(DiffMode mode) {
return mode == DiffMode::experimental_pushforward ||
mode == DiffMode::experimental_pullback ||
mode == DiffMode::experimental_vector_pushforward ||
mode == DiffMode::reverse_mode_forward_pass;
}
}

#endif
49 changes: 43 additions & 6 deletions include/clad/Differentiator/DiffPlanner.h
Original file line number Diff line number Diff line change
@@ -1,10 +1,11 @@
#ifndef CLAD_DIFF_PLANNER_H
#define CLAD_DIFF_PLANNER_H

#include "clad/Differentiator/DiffMode.h"
#include "clad/Differentiator/ParseDiffArgsTypes.h"
#include "clang/AST/RecursiveASTVisitor.h"
#include "llvm/ADT/SmallSet.h"
#include "clad/Differentiator/DiffMode.h"
#include "clad/Differentiator/DynamicGraph.h"
#include "clad/Differentiator/ParseDiffArgsTypes.h"

namespace clang {
class ASTContext;
Expand Down Expand Up @@ -90,9 +91,33 @@ struct DiffRequest {
/// 3) If no argument is provided, a default argument is used. The
/// function will be differentiated w.r.t. to its every parameter.
void UpdateDiffParamsInfo(clang::Sema& semaRef);

/// Define the == operator for DiffRequest.
bool operator==(const DiffRequest& other) const {
// either function match or previous declaration match
return (Function == other.Function ||
Function->getPreviousDecl() == other.Function ||
Function == other.Function->getPreviousDecl()) &&
BaseFunctionName == other.BaseFunctionName &&
CurrentDerivativeOrder == other.CurrentDerivativeOrder &&
RequestedDerivativeOrder == other.RequestedDerivativeOrder &&
CallContext == other.CallContext && Args == other.Args &&
Mode == other.Mode && EnableTBRAnalysis == other.EnableTBRAnalysis &&
DVI == other.DVI && use_enzyme == other.use_enzyme &&
DeclarationOnly == other.DeclarationOnly;
}

// String operator for printing the node.
operator std::string() const {
std::string res = BaseFunctionName + "__order_" +
std::to_string(CurrentDerivativeOrder) + "__mode_" +
DiffModeToString(Mode);
if (EnableTBRAnalysis)
res += "__TBR";
return res;
}
};

using DiffSchedule = llvm::SmallVector<DiffRequest, 16>;
using DiffInterval = std::vector<clang::SourceRange>;

struct RequestOptions {
Expand All @@ -106,9 +131,9 @@ struct DiffRequest {
///
DiffInterval& m_Interval;

/// The diff step-by-step plan for differentiation.
/// Graph to store the dependencies between different requests.
///
DiffSchedule& m_DiffPlans;
clad::DynamicGraph<DiffRequest>& m_DiffRequestGraph;

/// If set it means that we need to find the called functions and
/// add them for implicit diff.
Expand All @@ -120,12 +145,24 @@ struct DiffRequest {

public:
DiffCollector(clang::DeclGroupRef DGR, DiffInterval& Interval,
DiffSchedule& plans, clang::Sema& S, RequestOptions& opts);
clad::DynamicGraph<DiffRequest>& requestGraph, clang::Sema& S,
RequestOptions& opts);
bool VisitCallExpr(clang::CallExpr* E);

private:
bool isInInterval(clang::SourceLocation Loc) const;
};
}

// Define the hash function for DiffRequest.
template <> struct std::hash<clad::DiffRequest> {
std::size_t operator()(const clad::DiffRequest& DR) const {
// Use the function pointer as the hash of the DiffRequest, it
// is sufficient to break a reasonable number of collisions.
if (DR.Function->getPreviousDecl())
return std::hash<const void*>{}(DR.Function->getPreviousDecl());
return std::hash<const void*>{}(DR.Function);
}
};

#endif
1 change: 1 addition & 0 deletions include/clad/Differentiator/Differentiator.h
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@
#include "ArrayRef.h"
#include "BuiltinDerivatives.h"
#include "CladConfig.h"
#include "DynamicGraph.h"
#include "FunctionTraits.h"
#include "Matrix.h"
#include "NumericalDiff.h"
Expand Down
137 changes: 137 additions & 0 deletions include/clad/Differentiator/DynamicGraph.h
Original file line number Diff line number Diff line change
@@ -0,0 +1,137 @@
#ifndef CLAD_DIFFERENTIATOR_DYNAMICGRAPH_H
#define CLAD_DIFFERENTIATOR_DYNAMICGRAPH_H

#include <algorithm>
#include <functional>
#include <iostream>
#include <queue>
#include <set>
#include <unordered_map>
#include <unordered_set>
#include <vector>

namespace clad {
template <typename T> class DynamicGraph {
private:
/// Storing nodes in the graph. The index of the node in the vector is used as
/// a unique identifier for the node in the adjacency list.
std::vector<T> m_nodes;

/// Store the nodes in the graph as an unordered map from the node to a
/// boolean indicating whether the node is processed or not. The second
/// element in the pair is the id of the node in the nodes vector.
std::unordered_map<T, std::pair<bool, size_t>> m_nodeMap;

/// Store the adjacency list for the graph. The adjacency list is a map from
/// a node to the set of nodes that it has an edge to. We use integers inside
/// the set to avoid copying the nodes.
std::unordered_map<size_t, std::set<size_t>> m_adjList;

/// Set of source nodes in the graph.
std::set<size_t> m_sources;

/// Store the id of the node being processed right now.
int m_currentId = -1; // -1 means no node is being processed.

/// Maintain a queue of nodes to be processed next.
std::queue<size_t> m_toProcessQueue;

public:
DynamicGraph() = default;

/// Add an edge from the source node to the destination node.
/// \param src
/// \param dest
void addEdge(const T& src, const T& dest) {
std::pair<bool, size_t> srcInfo = addNode(src);
std::pair<bool, size_t> destInfo = addNode(dest);
size_t srcId = srcInfo.second;
size_t destId = destInfo.second;
m_adjList[srcId].insert(destId);
}

/// Add a node to the graph. If the node is already present, return the
/// id of the node in the graph. If the node is a source node, add it to the
/// queue of nodes to be processed.
/// \param node
/// \param isSource
/// \returns A pair of a boolean indicating whether the node is already
/// processed and the id of the node in the graph.
std::pair<bool, size_t> addNode(const T& node, bool isSource = false) {
if (m_nodeMap.find(node) == m_nodeMap.end()) {
size_t id = m_nodes.size();
m_nodes.push_back(node);
m_nodeMap[node] = {false, id}; // node is not processed yet.
m_adjList[id] = {};
if (isSource) {
m_sources.insert(id);
m_toProcessQueue.push(id);
}
}
return m_nodeMap[node];
}

/// Add an edge from the current node being processed to the
/// destination node.
/// \param dest
void addEdgeToCurrentNode(const T& dest) {
if (m_currentId == -1)
return;
addEdge(m_nodes[m_currentId], dest);
}

/// Set the current node being processed.
/// \param node
void setCurrentProcessingNode(const T& node) {
if (m_nodeMap.find(node) != m_nodeMap.end())
m_currentId = m_nodeMap[node].second;
}

/// Mark the current node being processed as processed and add the
/// destination nodes to the queue of nodes to be processed.
void markCurrentNodeProcessed() {
if (m_currentId != -1) {
m_nodeMap[m_nodes[m_currentId]].first = true;
for (size_t destId : m_adjList[m_currentId])
if (!m_nodeMap[m_nodes[destId]].first)
m_toProcessQueue.push(destId);
}
m_currentId = -1;
}

/// Get the nodes in the graph.
std::vector<T> getNodes() { return m_nodes; }

/// Print the nodes and edges in the graph.
void print() {
// First print the nodes with their insertion order.
for (const T& node : m_nodes) {
std::pair<bool, int> nodeInfo = m_nodeMap[node];
std::cout << (std::string)node << ": #" << nodeInfo.second;
if (m_sources.find(nodeInfo.second) != m_sources.end())
std::cout << " (source)";
if (nodeInfo.first)
std::cout << ", (done)\n";
else
std::cout << ", (unprocessed)\n";
}
// Then print the edges.
for (int i = 0; i < m_nodes.size(); i++)
for (size_t dest : m_adjList[i])
std::cout << i << " -> " << dest << "\n";
}

/// Get the next node to be processed from the queue of nodes to be
/// processed.
/// \returns The next node to be processed.
T getNextToProcessNode() {
if (m_toProcessQueue.empty())
return T();
size_t nextId = m_toProcessQueue.front();
m_toProcessQueue.pop();
return m_nodes[nextId];
}
};
} // end namespace clad

#endif // CLAD_DIFFERENTIATOR_DYNAMICGRAPH_H
2 changes: 1 addition & 1 deletion lib/Differentiator/BaseForwardModeVisitor.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1168,8 +1168,8 @@ StmtDiff BaseForwardModeVisitor::VisitCallExpr(const CallExpr* CE) {
// into the queue.
pushforwardFnRequest.DeclarationOnly = false;
pushforwardFnRequest.DerivedFDPrototype = pushforwardFD;
plugin::AddRequestToSchedule(m_CladPlugin, pushforwardFnRequest);
}
m_Builder.AddEdgeToGraph(pushforwardFnRequest);

if (pushforwardFD) {
if (baseDiff.getExpr()) {
Expand Down
37 changes: 29 additions & 8 deletions lib/Differentiator/DerivativeBuilder.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -37,8 +37,10 @@ using namespace clang;
namespace clad {

DerivativeBuilder::DerivativeBuilder(clang::Sema& S, plugin::CladPlugin& P,
const DerivedFnCollector& DFC)
const DerivedFnCollector& DFC,
clad::DynamicGraph<DiffRequest>& G)
: m_Sema(S), m_CladPlugin(P), m_Context(S.getASTContext()), m_DFC(DFC),
m_DiffRequestGraph(G),
m_NodeCloner(new utils::StmtClone(m_Sema, m_Context)),
m_BuiltinDerivativesNSD(nullptr), m_NumericalDiffNSD(nullptr) {}

Expand Down Expand Up @@ -292,15 +294,25 @@ static void registerDerivative(FunctionDecl* derivedFD, Sema& semaRef) {
assert(FD && "Must not be null.");
// If FD is only a declaration, try to find its definition.
if (!FD->getDefinition()) {
if (request.VerboseDiags)
diag(DiagnosticsEngine::Error,
request.CallContext ? request.CallContext->getBeginLoc() : noLoc,
"attempted differentiation of function '%0', which does not have a "
"definition", { FD->getNameAsString() });
return {};
// If only declaration is requested, allow this for non
// pullback/pushforward modes. For ex, this is required for Hessian -
// where we have forward mode followed by reverse mode, but we only need
// the declaration of the forward mode initially.
if (!request.DeclarationOnly ||
IsPullbackOrPushforwardMode(request.Mode)) {
if (request.VerboseDiags)
diag(DiagnosticsEngine::Error,
request.CallContext ? request.CallContext->getBeginLoc() : noLoc,
"attempted differentiation of function '%0', which does not "
"have a "
"definition",
{FD->getNameAsString()});
return {};
}
}

FD = FD->getDefinition();
if (!request.DeclarationOnly)
FD = FD->getDefinition();

// check if the function is non-differentiable.
if (clad::utils::hasNonDifferentiableAttribute(FD)) {
Expand Down Expand Up @@ -388,4 +400,13 @@ static void registerDerivative(FunctionDecl* derivedFD, Sema& semaRef) {
return DFI.DerivedFn();
return nullptr;
}

void DerivativeBuilder::AddEdgeToGraph(const DiffRequest& request) {
m_DiffRequestGraph.addEdgeToCurrentNode(request);
}

void DerivativeBuilder::AddEdgeToGraph(const DiffRequest& from,
const DiffRequest& to) {
m_DiffRequestGraph.addEdge(from, to);
}
}// end namespace clad
Loading

0 comments on commit b59c9aa

Please sign in to comment.