[PATCH] D73469: [ARM][AsmParser] Make assembly directives case insensitive

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 4 13:18:59 PST 2020


MaskRay added a comment.

Sorry, but I don't understand the rationale behind this change.

Yes, GNU as accepts `.Section` and llvm-mc doesn't. There are plenty of cases where MC does not recognize uppercase directives. I don't find this (`libgloss/arm/linux-syscalls0.S`) compelling enough to make MC accept every case-insensitive directive.
Changing the downstream projects may be more straightforward.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73469





More information about the llvm-commits mailing list