Use init_array_from helper for optimal codegen

Uses memcpy for trivially copyable types, element-wise copy otherwise.
ESPHOME_DEBUG_ASSERT catches size mismatches in integration tests.
This commit is contained in:
J. Nick Koston
2026-03-27 16:14:49 -10:00
parent 4cb1164a01
commit f8e144b23b
2 changed files with 16 additions and 5 deletions
+1 -5
View File
@@ -497,11 +497,7 @@ optional<float> calibrate_linear_compute(const std::array<float, 3> *functions,
template<size_t N> class CalibrateLinearFilter : public Filter {
public:
explicit CalibrateLinearFilter(std::initializer_list<std::array<float, 3>> linear_functions) {
ESPHOME_DEBUG_ASSERT(linear_functions.size() == N);
size_t i = 0;
for (const auto &f : linear_functions) {
this->linear_functions_[i++] = f;
}
init_array_from(this->linear_functions_, linear_functions);
}
optional<float> new_value(float value) override {
return calibrate_linear_compute(this->linear_functions_.data(), N, value);
+15
View File
@@ -497,6 +497,21 @@ template<typename T, size_t MAX_CAPACITY = std::numeric_limits<uint16_t>::max()>
index_type capacity_{0};
};
/// Initialize a std::array from an initializer_list. Uses memcpy for trivially copyable types (optimal codegen),
/// falls back to element-wise copy for non-trivially copyable types (e.g. TemplatableValue).
/// N is set by code generation; ESPHOME_DEBUG_ASSERT catches mismatches in debug/integration tests.
template<typename T, size_t N> inline void init_array_from(std::array<T, N> &dest, std::initializer_list<T> src) {
ESPHOME_DEBUG_ASSERT(src.size() == N);
if constexpr (std::is_trivially_copyable_v<T>) {
__builtin_memcpy(dest.data(), src.begin(), N * sizeof(T));
} else {
size_t i = 0;
for (const auto &v : src) {
dest[i++] = v;
}
}
}
/// Fixed-capacity vector - allocates once at runtime, never reallocates
/// This avoids std::vector template overhead (_M_realloc_insert, _M_default_append)
/// when size is known at initialization but not at compile time