[PATCH] D89561: [MC] Adjust StringTableBuilder for linked Mach-O binaries

Michael Trent via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 16 11:05:00 PDT 2020


mtrent added inline comments.


================
Comment at: llvm/lib/MC/StringTableBuilder.cpp:168
+  if (K == MachO || K == MachOLinked)
+    Size = alignTo(Size, 8); // Pad to multiple of 8.
+
----------------
This is only appropriate for 64-bit architectures. 32-bit architectures, such as arm64_32, should continue to use 4 byte alignment.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89561



More information about the llvm-commits mailing list