⚡ Bolt: [td_to_vakt optimizasyonu]#37
Conversation
…ildi - `td_to_vakt` fonksiyonunda gereksiz `std::to_string` çağrıları ve string birleştirme işlemleri, geçici bellek tahsislerini önlemek amacıyla sabit boyutlu bir karakter arabelleği (`char buf[]`) kullanılarak değiştirildi. - Bu optimizasyon, test sonuçlarına göre ilgili fonksiyonun yaklaşık 4-5 kat daha hızlı çalışmasını sağladı. - Önceki commit'ten kalan, `cached_nodes` ile ilgili derleme hatasına yol açan çakışan tanımlama düzeltilerek O(1) erişim mantığı tek bir fonksiyona (`cached_nodes_init` ve `cached_nodes_array`) entegre edildi. - `run_tests` binary dosyası kazara takip listesinden çıkarılarak (gitignore) ve disk üzerinden silinerek git temizliği yapıldı. Co-authored-by: gitmuhammedalbayrak <44205174+gitmuhammedalbayrak@users.noreply.github.com>
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
1 participant
💡 Ne yapıldı:
zaman::td_to_vaktiçindekistd::to_stringve+operatörü ile yapılan string oluşturma işlemleri kaldırılarak, sabit stack tahsisli (char buf[6]) basit aritmetik işlemler kullanan özel bir formata geçirildi.🎯 Neden:
Her zaman objesi yaratıldığında veya vakit formatı dönüşümü yapıldığında string birleştirme işlemleri arka planda gereksiz bellek tahsisleri (memory allocations) yaparak performans darboğazına sebep oluyordu. Bu durum mikro-optimizasyon fırsatı sunuyordu.
📊 Etki:
Testlere göre
td_to_vaktyaklaşık 4-5 kat daha hızlı çalışmaktadır (13.9 ms -> 3.0 ms for 100k iterations).🔬 Nasıl Ölçüldü:
Manuel olarak bir
std::chronotabanlı benchmark senaryosunda 100 bin kere her iki fonksiyon çağrılarak süre farkı kontrol edildi. Tümtest_zaman.cpptestleri başarılı oldu, çıktı düzenleri değişmedi. İşleyiş doğruluğu onaylandı.PR created automatically by Jules for task 7910582020881019106 started by @gitmuhammedalbayrak