Anakin100100 opened a new pull request, #49298:
URL: https://github.com/apache/arrow/pull/49298

   ### Rationale for this change
   Before C++ 20 there was no built in implementation for many common bit 
operations utilities included in the stdlib so they were implemented in 
bit_utils.h. Now that they are included in the stdlib they should be removed 
from bit_utils to decrease the amount of code that needs to be maintained as 
described in #48591 
   
   ### What changes are included in this PR?
   IsPowerOf2, PopCount, CountLeadingZeros, CountTrailingZeros, NumRequiredBits 
are removed from bit_utils and replaced with their equivalents from bit.h i.e. 
has_single_bit, popcount, countl_zero, countr_zero and bit_width. 
   
   ### Are these changes tested?
   No new code is introduced and the stdlib implementation maintains parity 
with the replaced functions so no new unit tests are necessary. 
   
   ### Are there any user-facing changes?
   No


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to