Skip to content

Commit 85ab9c2

Browse files
CharryWuLegNeato
authored andcommitted
fix(cudnn): remove duplicate ConvBwdData match arm in backend graph builder
The operations iterator map in GraphBuilder::build() had two identical Operation::ConvBwdData arms. The second arm was unreachable dead code. Removed the duplicate; all seven operation variants are now matched once. Made-with: Cursor
1 parent 5e49cf3 commit 85ab9c2

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

crates/cudnn/src/backend/graph.rs

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
use crate::{
1+
use crate::{
22
CudnnContext, CudnnError,
33
backend::{Descriptor, Operation},
44
};
@@ -39,7 +39,6 @@ impl GraphBuilder {
3939
let descriptors = operations
4040
.iter()
4141
.map(|op| match op {
42-
Operation::ConvBwdData { raw, .. } => raw.inner(),
4342
Operation::ConvBwdData { raw, .. } => raw.inner(),
4443
Operation::ConvBwdFilter { raw, .. } => raw.inner(),
4544
Operation::ConvFwd { raw, .. } => raw.inner(),

0 commit comments

Comments
 (0)