Hi,<div><br></div><div>I took a look into &quot;channel.hpp&quot; and found that packed_channel_value is not very efficient in construction.</div><div>There are:</div><div><br></div><div>    packed_channel_value(integer_t v) { _value = static_cast&lt; integer_t &gt;( v % num_values ); }</div>
<div><br></div><div>    // I have no idea why this exists, seems OK to remove it</div><div><div>    template &lt;typename Scalar&gt; packed_channel_value(Scalar v) { _value = static_cast&lt; integer_t &gt;( v ) % num_values; }</div>
</div><div><br></div><div>IIUC,  &quot;v % num_values&quot; above could be replaced with a much cheaper &quot;v &amp; low_bits_mask_t&lt;NumBits&gt;::sig_bits_fast&quot;.</div><div>That is:</div><div><br></div><div>    packed_channel_value(integer_t v) : _value(v &amp; low_bits_mask_t&lt;NumBits&gt;::sig_bits_fast) {}</div>
<div><br></div><div>What do you think? Am I missing some point?</div><div><br></div><div><br></div><div>Thanks.</div><div><br></div>