[PATCH] D67486: [Alignment] Introduce llvm::Align to MCSection

[Clement Courbet via Phabricator via llvm-commits]

courbet added inline comments.


================
Comment at: llvm/lib/MC/MachObjectWriter.cpp:243
   W.write<uint32_t>(FileOffset);
-
-  assert(isPowerOf2_32(Section.getAlignment()) && "Invalid alignment!");
-  W.write<uint32_t>(Log2_32(Section.getAlignment()));
+  W.write<uint32_t>(Log2(Section.getAlignment()));
   W.write<uint32_t>(NumRelocations ? RelocationsStart : 0);
----------------
Oh, and this calls `Log2(double)`.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D67486/new/

https://reviews.llvm.org/D67486