From f50550f4ce3981862d9397ac05015232350416a5 Mon Sep 17 00:00:00 2001 From: Jan Wassenberg Date: Mon, 15 Dec 2025 02:40:45 -0800 Subject: [PATCH] Warning fixes (sign mismatch), switch default PiperOrigin-RevId: 844679375 --- gemma/configs.h | 1 + gemma/gemma.cc | 2 +- gemma/query.h | 2 +- 3 files changed, 3 insertions(+), 2 deletions(-) diff --git a/gemma/configs.h b/gemma/configs.h index 38fab43f..b7274801 100644 --- a/gemma/configs.h +++ b/gemma/configs.h @@ -107,6 +107,7 @@ static inline int AttentionImplToFlags(AttentionImpl impl, case AttentionImpl::kOld: return kAttentionUseOld; case AttentionImpl::kFlash: + default: return 0; } } diff --git a/gemma/gemma.cc b/gemma/gemma.cc index 055caaeb..c58a5a87 100644 --- a/gemma/gemma.cc +++ b/gemma/gemma.cc @@ -628,7 +628,7 @@ static void GenerateTWithContinuousBatching( const SampleFunc sample_token = ChooseSampleFunc(runtime_config, engine, env.ctx); - int query_inserted = 0; + size_t query_inserted = 0; while (non_eos.Any() || query_inserted < all_queries.NumQueries()) { for (size_t qi = 0; qi < qbatch.Size(); ++qi) { // Continue if qi slot is still processing. diff --git a/gemma/query.h b/gemma/query.h index 9c50a062..36e8ee5c 100644 --- a/gemma/query.h +++ b/gemma/query.h @@ -131,7 +131,7 @@ class QBatch { HWY_DASSERT(size_ != 0); HWY_DASSERT(start_ + size_ <= queries_.NumQueries()); query_idx_.reserve(size_); - for (int i = 0; i < size_; ++i) { + for (size_t i = 0; i < size_; ++i) { query_idx_.push_back(start_ + i); } }