Skip to content

Commit b24c0bb

Browse files
committed
Remove unused batched-LDE wrapper functions
coset_lde_batch_base_into_with_merkle_tree, coset_lde_batch_ext3_into_with_merkle_tree, and coset_lde_batch_ext3_into_with_leaf_hash have no callers anywhere (only their own definitions + an assert-message string). They are thin non-keep wrappers over the *_inner functions, which stay live via the *_keep variants (the R1 GPU path, called from stark::gpu_lde) and coset_lde_batch_base_into_with_leaf_hash. Pure deletion, -75 lines.
1 parent 38feb15 commit b24c0bb

1 file changed

Lines changed: 0 additions & 75 deletions

File tree

crypto/math-cuda/src/lde.rs

Lines changed: 0 additions & 75 deletions
Original file line numberDiff line numberDiff line change
@@ -617,33 +617,6 @@ pub fn coset_lde_batch_base_into_with_leaf_hash(
617617
.map(|_| ())
618618
}
619619

620-
/// Like `coset_lde_batch_base_into_with_leaf_hash`, but also builds the full
621-
/// Merkle tree on device and returns the `2*lde_size - 1` node buffer back
622-
/// to the caller in `merkle_nodes_out` (byte length `(2*lde_size - 1) * 32`).
623-
///
624-
/// The leaf hashes are never exposed to the caller — they stay on device and
625-
/// feed straight into the pair-hash tree kernel, avoiding the
626-
/// pinned→pageable→pinned round-trip that the separate-step GPU tree build
627-
/// would pay.
628-
pub fn coset_lde_batch_base_into_with_merkle_tree(
629-
columns: &[&[u64]],
630-
blowup_factor: usize,
631-
weights: &[u64],
632-
outputs: &mut [&mut [u64]],
633-
merkle_nodes_out: &mut [u8],
634-
) -> Result<()> {
635-
coset_lde_batch_base_into_with_merkle_tree_inner(
636-
columns,
637-
blowup_factor,
638-
weights,
639-
outputs,
640-
merkle_nodes_out,
641-
KeccakCommit::FullTree,
642-
false,
643-
)
644-
.map(|_| ())
645-
}
646-
647620
/// Fused LDE + leaf-hash + Merkle tree build. If `keep_device_buf` is true,
648621
/// returns an `Arc<CudaSlice<u64>>` wrapping the LDE device buffer so callers
649622
/// (R2–R4 GPU paths) can reuse the LDE without a re-H2D.
@@ -828,54 +801,6 @@ fn coset_lde_batch_base_into_with_merkle_tree_inner(
828801
}
829802
}
830803

831-
/// Ext3 variant of `coset_lde_batch_base_into_with_leaf_hash`: fused
832-
/// LDE + Keccak-256 leaf hashing over ext3 columns. Thin wrapper over
833-
/// `coset_lde_batch_ext3_into_with_merkle_tree_inner` with `LeavesOnly`.
834-
pub fn coset_lde_batch_ext3_into_with_leaf_hash(
835-
columns: &[&[u64]],
836-
n: usize,
837-
blowup_factor: usize,
838-
weights: &[u64],
839-
outputs: &mut [&mut [u64]],
840-
hashed_leaves_out: &mut [u8],
841-
) -> Result<()> {
842-
coset_lde_batch_ext3_into_with_merkle_tree_inner(
843-
columns,
844-
n,
845-
blowup_factor,
846-
weights,
847-
outputs,
848-
hashed_leaves_out,
849-
KeccakCommit::LeavesOnly,
850-
false,
851-
)
852-
.map(|_| ())
853-
}
854-
855-
/// Ext3 variant of the fused `coset_lde_batch_base_into_with_merkle_tree`.
856-
/// LDE + leaf hashing + inner-tree build, all on device; D2Hs only the LDE
857-
/// evaluations and the full `2*lde_size - 1` node buffer.
858-
pub fn coset_lde_batch_ext3_into_with_merkle_tree(
859-
columns: &[&[u64]],
860-
n: usize,
861-
blowup_factor: usize,
862-
weights: &[u64],
863-
outputs: &mut [&mut [u64]],
864-
merkle_nodes_out: &mut [u8],
865-
) -> Result<()> {
866-
coset_lde_batch_ext3_into_with_merkle_tree_inner(
867-
columns,
868-
n,
869-
blowup_factor,
870-
weights,
871-
outputs,
872-
merkle_nodes_out,
873-
KeccakCommit::FullTree,
874-
false,
875-
)
876-
.map(|_| ())
877-
}
878-
879804
/// Ext3 variant of [`coset_lde_batch_base_into_with_merkle_tree_keep`] —
880805
/// returns an `Arc<CudaSlice<u64>>` handle to the de-interleaved LDE device
881806
/// buffer.

0 commit comments

Comments
 (0)