[BOLT][NFC] Rename getNames for PLT, TailDuplication

[paschalis-mpeis]

No description provided.

[llvmbot]

@llvm/pr-subscribers-bolt

Author: Paschalis Mpeis (paschalis-mpeis)

Changes

---

Full diff: https://github.com/llvm/llvm-project/pull/119870.diff

2 Files Affected:

• (modified) bolt/include/bolt/Passes/PLTCall.h (+1-1)
• (modified) bolt/include/bolt/Passes/TailDuplication.h (+1-1)

diff --git a/bolt/include/bolt/Passes/PLTCall.h b/bolt/include/bolt/Passes/PLTCall.h
index 09ef96e27293da..9c46f5ddf701cc 100644
--- a/bolt/include/bolt/Passes/PLTCall.h
+++ b/bolt/include/bolt/Passes/PLTCall.h
@@ -26,7 +26,7 @@ class PLTCall : public BinaryFunctionPass {
   explicit PLTCall(const cl::opt<bool> &PrintPass)
       : BinaryFunctionPass(PrintPass) {}
 
-  const char *getName() const override { return "PLT call optimization"; }
+  const char *getName() const override { return "plt-call-optimization"; }
   bool shouldPrint(const BinaryFunction &BF) const override {
     return BinaryFunctionPass::shouldPrint(BF);
   }
diff --git a/bolt/include/bolt/Passes/TailDuplication.h b/bolt/include/bolt/Passes/TailDuplication.h
index a2fcab0720ca27..4a7ec083bc4857 100644
--- a/bolt/include/bolt/Passes/TailDuplication.h
+++ b/bolt/include/bolt/Passes/TailDuplication.h
@@ -143,7 +143,7 @@ class TailDuplication : public BinaryFunctionPass {
 
   explicit TailDuplication() : BinaryFunctionPass(false) {}
 
-  const char *getName() const override { return "tail duplication"; }
+  const char *getName() const override { return "tail-duplication"; }
 
   Error runOnFunctions(BinaryContext &BC) override;
 };

[paschalis-mpeis]

Minor patch to use the same convention on TailDuplication and PLTCall passes.
Also avoids creating filenames with spaces when printing those functions.

[bgergely0]

LGTM!

[paschalis-mpeis]

Thanks Gergely. I forgot about this patch!