Skip to content

Commit f3c5e7d

Browse files
alexmalyshevfacebook-github-bot
authored andcommitted
Use JIT_CHECK in CopyGraph::addEdge
Summary: This exposes the crash with test_xml_etree. This JIT_DCHECK is scary because if we fail it we'll crash later in the program's execution, where it's harder to debug. Reviewed By: yoney, jbower-fb Differential Revision: D81512871 fbshipit-source-id: ec0c0677e0dda548f94d081b9716f8651db57892
1 parent 6a48d26 commit f3c5e7d

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

cinderx/Jit/codegen/copy_graph.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ CopyGraph::Node::~Node() {
1616
void CopyGraph::addEdge(int from, int to) {
1717
auto parent = getNode(from);
1818
auto child = getNode(to);
19-
JIT_DCHECK(child->parent == nullptr, "child already has a parent");
19+
JIT_CHECK(child->parent == nullptr, "child already has a parent");
2020
setParent(child, parent);
2121
}
2222

0 commit comments

Comments
 (0)