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

[Peter Smith via Phabricator via llvm-commits]

peter.smith added a comment.

No objections as I think a defaut alignment of 2 for Thumb is the right thing to do.

One thing that may cause problems is an immediate inline change of state to Arm something like the following with a thumb target.

  .text
  .arm
  nop

Whereas the default alignment of 4 would save us beforehand. There is a small risk that some existing projects could be affected. I think that an eventual fix for pr/51929 could address this case though. Such projects could also add an align directive as well.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110580