[PATCH] D16549: [MC] Use .p2align instead of .align

[Dan Gohman via llvm-commits]

sunfish created this revision.
sunfish added subscribers: majnemer, compnerd, joerg, llvm-commits.
sunfish set the repository for this revision to rL LLVM.
Herald added a reviewer: tstellarAMD.
Herald added subscribers: dsanders, dschuff, srhines, danalbert, jfb, qcolombet, MatzeB, tberghammer.

For historic reasons, the behavior of .align differs between targets. Fortunately, there are alternatives, .p2align and .balign, which make the interpretation of the parameter explicit, and which behave consistently across targets.

This patch teaches MC to use .p2align instead of .align, so that people reading code for multiple architectures don't have to remember which way each platform does its .align directive.


Repository:
  rL LLVM

http://reviews.llvm.org/D16549

Files:
  lib/MC/MCAsmStreamer.cpp
  test/CodeGen/AArch64/arm64-swizzle-tbl-i16-layout.ll
  test/CodeGen/AArch64/arm64-tls-dynamic-together.ll
  test/CodeGen/AArch64/emutls.ll
  test/CodeGen/AArch64/emutls_generic.ll
  test/CodeGen/AArch64/global-merge-3.ll
  test/CodeGen/AArch64/stackmap-liveness.ll
  test/CodeGen/AMDGPU/elf.ll
  test/CodeGen/ARM/Windows/long-calls.ll
  test/CodeGen/ARM/align.ll
  test/CodeGen/ARM/byval_load_align.ll
  test/CodeGen/ARM/ehabi-handlerdata-nounwind.ll
  test/CodeGen/ARM/ehabi-handlerdata.ll
  test/CodeGen/ARM/emutls_generic.ll
  test/CodeGen/ARM/globals.ll
  test/CodeGen/ARM/memfunc.ll
  test/CodeGen/ARM/preferred-align.ll
  test/CodeGen/ARM/thumb-alignment.ll
  test/CodeGen/ARM/tls-models.ll
  test/CodeGen/ARM/tls3.ll
  test/CodeGen/Mips/2010-07-20-Switch.ll
  test/CodeGen/Mips/cconv/memory-layout.ll
  test/CodeGen/Mips/const4a.ll
  test/CodeGen/Mips/const6.ll
  test/CodeGen/Mips/const6a.ll
  test/CodeGen/Mips/ehframe-indirect.ll
  test/CodeGen/Mips/emutls_generic.ll
  test/CodeGen/Mips/lcb5.ll
  test/CodeGen/Mips/longbranch.ll
  test/CodeGen/Mips/nacl-align.ll
  test/CodeGen/PowerPC/2010-02-04-EmptyGlobal.ll
  test/CodeGen/PowerPC/align.ll
  test/CodeGen/PowerPC/code-align.ll
  test/CodeGen/PowerPC/ppc64-linux-func-size.ll
  test/CodeGen/PowerPC/ppc64-toc.ll
  test/CodeGen/Thumb/2010-07-01-FuncAlign.ll
  test/CodeGen/Thumb/segmented-stacks.ll
  test/CodeGen/Thumb2/aligned-constants.ll
  test/CodeGen/Thumb2/thumb2-tbb.ll
  test/CodeGen/WebAssembly/global.ll
  test/CodeGen/X86/GC/dynamic-frame-size.ll
  test/CodeGen/X86/GC/erlang-gc.ll
  test/CodeGen/X86/GC/ocaml-gc.ll
  test/CodeGen/X86/alignment.ll
  test/CodeGen/X86/avoid-loop-align.ll
  test/CodeGen/X86/avx2-vbroadcast.ll
  test/CodeGen/X86/block-placement.ll
  test/CodeGen/X86/bss_pagealigned.ll
  test/CodeGen/X86/cfstring.ll
  test/CodeGen/X86/code_placement_align_all.ll
  test/CodeGen/X86/constructor.ll
  test/CodeGen/X86/emutls_generic.ll
  test/CodeGen/X86/global-sections.ll
  test/CodeGen/X86/licm-symbol.ll
  test/CodeGen/X86/osx-private-labels.ll
  test/CodeGen/X86/pic.ll
  test/CodeGen/X86/postra-licm.ll
  test/CodeGen/X86/regalloc-reconcile-broken-hints.ll
  test/CodeGen/X86/seh-catch-all-win32.ll
  test/CodeGen/X86/stackmap-large-constants.ll
  test/CodeGen/X86/stackmap-liveness.ll
  test/CodeGen/X86/statepoint-allocas.ll
  test/CodeGen/X86/statepoint-invoke.ll
  test/CodeGen/X86/statepoint-stackmap-format.ll
  test/CodeGen/X86/tls-android.ll
  test/CodeGen/X86/unaligned-load.ll
  test/CodeGen/X86/utf16-cfstrings.ll
  test/CodeGen/X86/win-cleanuppad.ll
  test/CodeGen/X86/win32-eh.ll
  test/CodeGen/X86/win_cst_pool.ll
  test/CodeGen/XCore/align.ll
  test/CodeGen/XCore/epilogue_prologue.ll
  test/CodeGen/XCore/scavenging.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D16549.45898.patch
Type: text/x-patch
Size: 59655 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160125/794b74be/attachment.bin>