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

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 25 09:57:33 PDT 2019


MaskRay added inline comments.


================
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;
----------------
peter.smith wrote:
> 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)
@jcai19 You can run `git diff -U0 --no-color 'HEAD^' | ~/llvm-project/clang/tools/clang-format/clang-format-diff.py -i -p1` to format the comment.


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