From 237970b25903a45cb8748cd41ffa05d0a161a8a9 Mon Sep 17 00:00:00 2001
From: Zhang Junyu <junyu92@gmail.com>
Date: Sat, 4 Nov 2023 23:29:49 +0800
Subject: [PATCH] perf: reduce unnecessary clone

---
 crates/zkwasm/src/circuits/utils/table_entry.rs | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)

diff --git a/crates/zkwasm/src/circuits/utils/table_entry.rs b/crates/zkwasm/src/circuits/utils/table_entry.rs
index 5baf7a8c3..166b8552d 100644
--- a/crates/zkwasm/src/circuits/utils/table_entry.rs
+++ b/crates/zkwasm/src/circuits/utils/table_entry.rs
@@ -114,19 +114,19 @@ pub struct MemoryRWEntry {
 }
 
 #[derive(Debug)]
-pub struct EventTableEntryWithMemoryInfo {
-    pub eentry: EventTableEntry,
+pub struct EventTableEntryWithMemoryInfo<'a> {
+    pub eentry: &'a EventTableEntry,
     pub memory_rw_entires: Vec<MemoryRWEntry>,
 }
 
 #[derive(Debug)]
-pub(crate) struct EventTableWithMemoryInfo(
-    pub(in crate::circuits) Vec<EventTableEntryWithMemoryInfo>,
+pub(crate) struct EventTableWithMemoryInfo<'a>(
+    pub(in crate::circuits) Vec<EventTableEntryWithMemoryInfo<'a>>,
 );
 
-impl EventTableWithMemoryInfo {
+impl<'a> EventTableWithMemoryInfo<'a> {
     pub(in crate::circuits) fn new(
-        event_table: &EventTable,
+        event_table: &'a EventTable,
         memory_writing_table: &MemoryWritingTable,
     ) -> Self {
         let lookup = memory_writing_table.build_lookup_mapping();
@@ -140,6 +140,7 @@ impl EventTableWithMemoryInfo {
                     .unwrap();
                 records[idx]
             } else {
+                println!("lookup eid: {}", eid);
                 let idx = records
                     .binary_search_by(|(start_eid, end_eid)| {
                         if eid <= start_eid {
@@ -160,7 +161,7 @@ impl EventTableWithMemoryInfo {
                 .entries()
                 .iter()
                 .map(|eentry| EventTableEntryWithMemoryInfo {
-                    eentry: eentry.clone(),
+                    eentry: &eentry,
                     memory_rw_entires: memory_event_of_step(eentry, &mut 1)
                         .iter()
                         .map(|mentry| {