[PATCH] D110580: [THUMB2] default .text alignment to 2B

Nick Desaulniers via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 27 13:21:06 PDT 2021


nickdesaulniers created this revision.
nickdesaulniers added reviewers: peter.smith, rengolin, MaskRay.
Herald added subscribers: JDevlieghere, hiraditya, kristof.beyls, mgorny, emaste.
nickdesaulniers requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

Similar to GAS, default the alignment of .text to 2B. This difference
between clang vs GAS was frustrating efforts to debug pr/51929.

Similar to D102052 <https://reviews.llvm.org/D102052>.

Does not regress Linux kernel boots in QEMU. Tested with Linux kernel
tag v5.15-rc3 with
https://www.arm.linux.org.uk/developer/patches/viewpatch.php?id=9122/1
applied, and QEMU tag v5.2.0. In pr/51929 we're working to dig our way
out of boot failures with ToT QEMU (v6.1.0+).

Link: https://github.com/ClangBuiltLinux/linux/issues/1447


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D110580

Files:
  lld/test/ELF/arm-thumb-branch.s
  lld/test/ELF/arm-thunk-linkerscript-orphan.s
  llvm/lib/Target/ARM/MCTargetDesc/ARMMCObjectFileInfo.cpp
  llvm/lib/Target/ARM/MCTargetDesc/ARMMCObjectFileInfo.h
  llvm/lib/Target/ARM/MCTargetDesc/ARMMCTargetDesc.cpp
  llvm/lib/Target/ARM/MCTargetDesc/CMakeLists.txt
  llvm/utils/gn/secondary/llvm/lib/Target/ARM/MCTargetDesc/BUILD.gn

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D110580.375384.patch
Type: text/x-patch
Size: 6696 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210927/aa9faeff/attachment.bin>


More information about the llvm-commits mailing list