[PATCH] D35545: [AArch64, COFF] Interpret .align as power of two for COFF as well

[Martin Storsjö via Phabricator via llvm-commits]

mstorsjo updated this revision to Diff 107165.
mstorsjo added a comment.

Updated to a completely new test that explicitly checks the alignment.


https://reviews.llvm.org/D35545

Files:
  lib/Target/AArch64/MCTargetDesc/AArch64MCAsmInfo.cpp
  test/MC/COFF/aarch64-align.s


Index: test/MC/COFF/aarch64-align.s
===================================================================
--- /dev/null
+++ test/MC/COFF/aarch64-align.s
@@ -0,0 +1,7 @@
+// RUN: llvm-mc -filetype=obj -triple aarch64-windows-gnu %s | llvm-readobj -s -sd | FileCheck %s
+    .text
+    .align 5
+f0:
+    ret
+
+// CHECK: IMAGE_SCN_ALIGN_32BYTES
Index: lib/Target/AArch64/MCTargetDesc/AArch64MCAsmInfo.cpp
===================================================================
--- lib/Target/AArch64/MCTargetDesc/AArch64MCAsmInfo.cpp
+++ lib/Target/AArch64/MCTargetDesc/AArch64MCAsmInfo.cpp
@@ -105,4 +105,5 @@
   CommentString = ";";
   PrivateGlobalPrefix = ".L";
   PrivateLabelPrefix = ".L";
+  AlignmentIsInBytes = false;
 }


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D35545.107165.patch
Type: text/x-patch
Size: 723 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170718/1c1471d7/attachment.bin>