[PATCH] D69296: [ARM] Uses "Sun Style" syntax for section switching

Peter Smith via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 25 02:01:21 PDT 2019


peter.smith accepted this revision.
peter.smith added a comment.
This revision is now accepted and ready to land.

Thanks for the update. Looks good to me. Please can you apply the formatting change to the comment (80 columns) prior to commit.



================
Comment at: llvm/lib/Target/ARM/MCTargetDesc/ARMMCAsmInfo.cpp:79
+
+  // The GNU assembler supports Sun style section switching for Arm targets, and it is used in projects like the Linux kernel.
+  SunStyleELFSectionSwitchSyntax = true;
----------------
Can you break this comment into two lines so that it doesn't go over 80 columns? (https://llvm.org/docs/CodingStandards.html#source-code-width)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69296





More information about the llvm-commits mailing list