[PATCH] D118732: [BOLT] Add aarch64 support for peephole passes

Vladislav Khmelevsky via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 2 07:28:48 PST 2022


yota9 marked an inline comment as done.
yota9 added inline comments.


================
Comment at: bolt/lib/Passes/BinaryPasses.cpp:110
+    "peepholes", cl::CommaSeparated, cl::desc("enable peephole optimizations"),
+    cl::value_desc("opt1,opt2,opt3,..."),
+    cl::values(clEnumValN(Peepholes::PEEP_NONE, "none", "disable peepholes"),
----------------
rafauler wrote:
> I'm not sure if this first character that phabricator is displaying here means there is a \t character in the source code. If there is, you should configure your editor to do not insert tabs and use only whitespaces (as per https://llvm.org/docs/CodingStandards.html#id18).
First time I saw that I was thinking it is tab too, but it's just space, idented by clang-format


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118732



More information about the llvm-commits mailing list