GlcoinCore/clboss
1
0
Fork 0
mirror of https://github.com/ZmnSCPxj/clboss.git synced 2026-08-13 12:33:20 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 5

fix: initialize ReservoirSampler::wsum to silence build warning

Browse source 
GCC's deeper inlining at -O2 detects that wsum may be used uninitialized when
ReservoirSampler::add() is inlined into JitRebalancer. Debug builds (-Og) don't
trigger this because GCC doesn't inline deeply enough. The value is overwritten
on the first add() call, so this is purely to satisfy the compiler.
This commit is contained in:
Ken Sedgwick 2026-03-04 11:10:33 -08:00
parent a17b83ce2d
commit d90ca280ef
No known key found for this signature in database
GPG key ID: DBD2AF0849D711A9
1 changed files with 2 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

4
Stats/ReservoirSampler.hpp
View file

@ -26,10 +26,10 @@ private:
Weight wsum;
public:
ReservoirSampler() : max_selected(1) { }
ReservoirSampler() : max_selected(1), wsum(0) { }
explicit
ReservoirSampler(std::size_t max_selected_
) : max_selected(max_selected_) { }
) : max_selected(max_selected_), wsum(0) { }
ReservoirSampler(ReservoirSampler&&) =default;
ReservoirSampler(ReservoirSampler const&) =default;