[PATCH] D130824: [BOLT] Fixed some typos

Amir Ayupov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jul 30 11:40:58 PDT 2022


Amir accepted this revision.
Amir added inline comments.
This revision is now accepted and ready to land.


================
Comment at: bolt/lib/Passes/RetpolineInsertion.cpp:47
 
-cl::opt<RetpolineInsertion::AvailabilityOptions>
-R11Availability("r11-availability",
-  cl::desc("determine the availablity of r11 before indirect branches"),
-  cl::init(RetpolineInsertion::AvailabilityOptions::NEVER),
-  cl::values(
-    clEnumValN(RetpolineInsertion::AvailabilityOptions::NEVER,
-      "never", "r11 not available"),
-    clEnumValN(RetpolineInsertion::AvailabilityOptions::ALWAYS,
-      "always", "r11 avaialable before calls and jumps"),
-    clEnumValN(RetpolineInsertion::AvailabilityOptions::ABI,
-      "abi", "r11 avaialable before calls but not before jumps")),
-  cl::ZeroOrMore,
-  cl::cat(BoltCategory));
+cl::opt<RetpolineInsertion::AvailabilityOptions> R11Availability(
+    "r11-availability",
----------------
GabrielRavier wrote:
> Amir wrote:
> > I don't see any typos fixed here, can you please revert this hunk?
> The fixed typo is "availablity" (which is spelled availability from what I can see), otherwise the formatting in general is caused by the `git clang-format` command that the LLVM documentation implies is required when making a patch.
Sorry, you’re right.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130824



More information about the llvm-commits mailing list