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

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 28 15:33:50 PDT 2021


craig.topper added a comment.

In D110660#3028806 <https://reviews.llvm.org/D110660#3028806>, @nickdesaulniers wrote:

> (oh, this is actually good enough to fix pr/51929). Maybe I can add a fixme to the test on the Size noting that GAS adds explicit nops. We don't necessarily need the nops; we need to not treat `.align 0` as `.align 1` on THUMB2 (and probably ARM; unverified).

Is this an issue for RISCV too?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110660



More information about the llvm-commits mailing list