[PATCH] D110660: [AsmParser] handle .align 0 per MCAsmInfo

Tue Sep 28 19:20:14 PDT 2021

MaskRay added a comment.

> While .align 0 may be converted to .balign 1 for x86_64 elf, on THUMB2 it's converted to .balign 4.

Does gas do so? This is a surprising behavior. I hope we don't follow. The user should be fixed instead.

https://sourceware.org/binutils/docs/as/Align.html#Align says a default value is used. If thumb typically uses 2-byte alignment, `.align 0` meaning `.balign 4` seems unexpected.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110660