[PATCH] D139181: [lld][Alignment][NFC] Use Align instead of log2 of alignment in Wasm Sections

Alexander Richardson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 2 07:27:18 PST 2022


arichardson added inline comments.


================
Comment at: lld/wasm/InputChunks.h:88
   uint32_t inputSectionOffset = 0;
-  uint32_t alignment;
+  llvm::Align alignment;
   uint32_t flags;
----------------
Reordering the fields would reduce padding.


================
Comment at: lld/wasm/OutputSegment.h:41
   uint32_t sectionOffset = 0;
-  uint32_t alignment = 0;
+  llvm::Align alignment;
   uint64_t startVA = 0;
----------------
Same here


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139181



More information about the llvm-commits mailing list