commit 4f0cdcd0beba315176c01a38dbf7b1ed46fdbd55
author Kevin Lubick <kjlubick@users.noreply.github.com> Mon Jan 09 19:22:33 2023 -0600
committer GitHub <noreply@github.com> Mon Jan 09 17:22:33 2023 -0800
tree 962de80e5f358d055b8e850ea3f858ba84111e5f
parent b6720a5b7b8dfafae04938f488e412afbb37bd65

Inline usage of SkIsPow2 (#38722)

* Inline usage of SkIsPow2

* static

display_list/display_list_builder.cc

diff --git a/display_list/display_list_builder.cc b/display_list/display_list_builder.cc
index b5b1c0e..b091e6d 100644
--- a/display_list/display_list_builder.cc
+++ b/display_list/display_list_builder.cc
@@ -26,12 +26,16 @@
   CopyV(SkTAddOffset<void>(dst, n * sizeof(S)), std::forward<Rest>(rest)...);
 }
 
+static constexpr inline bool is_power_of_two(int value) {
+  return (value & (value - 1)) == 0;
+}
+
 template <typename T, typename... Args>
 void* DisplayListBuilder::Push(size_t pod, int render_op_inc, Args&&... args) {
   size_t size = SkAlignPtr(sizeof(T) + pod);
   FML_DCHECK(size < (1 << 24));
   if (used_ + size > allocated_) {
-    static_assert(SkIsPow2(DL_BUILDER_PAGE),
+    static_assert(is_power_of_two(DL_BUILDER_PAGE),
                   "This math needs updating for non-pow2.");
     // Next greater multiple of DL_BUILDER_PAGE.
     allocated_ = (used_ + size + DL_BUILDER_PAGE) & ~(DL_BUILDER_PAGE - 1);