[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