[llvm] [RISCV] Simplify PrintExtension. NFC (PR #75427)
Craig Topper via llvm-commits
llvm-commits at lists.llvm.org
Wed Dec 13 21:21:57 PST 2023
https://github.com/topperc created https://github.com/llvm/llvm-project/pull/75427
Instead of using a format string that needs to be parsed, we can use left_justify to print each string with padding.
>From 72bc0c89e5c6129819cc83ac41b0bd6ee419a759 Mon Sep 17 00:00:00 2001
From: Craig Topper <craig.topper at sifive.com>
Date: Wed, 13 Dec 2023 21:19:32 -0800
Subject: [PATCH] [RISCV] Simplify PrintExtension. NFC
Instead of using a format string that needs to be parsed, we can
use left_justify to print each string with padding.
---
llvm/lib/Support/RISCVISAInfo.cpp | 15 ++++++++-------
1 file changed, 8 insertions(+), 7 deletions(-)
diff --git a/llvm/lib/Support/RISCVISAInfo.cpp b/llvm/lib/Support/RISCVISAInfo.cpp
index 85c34dd6206307..245d1e67343a97 100644
--- a/llvm/lib/Support/RISCVISAInfo.cpp
+++ b/llvm/lib/Support/RISCVISAInfo.cpp
@@ -215,11 +215,12 @@ static void verifyTables() {
#endif
}
-static void PrintExtension(const std::string Name, const std::string Version,
- const std::string Description) {
- outs() << " "
- << format(Description.empty() ? "%-20s%s\n" : "%-20s%-10s%s\n",
- Name.c_str(), Version.c_str(), Description.c_str());
+static void PrintExtension(StringRef Name, StringRef Version,
+ StringRef Description) {
+ outs().indent(4);
+ unsigned VersionWidth = Description.empty() ? 0 : 10;
+ outs() << left_justify(Name, 20) << left_justify(Version, VersionWidth)
+ << Description << "\n";
}
void llvm::riscvExtensionsHelp(StringMap<StringRef> DescMap) {
@@ -233,7 +234,7 @@ void llvm::riscvExtensionsHelp(StringMap<StringRef> DescMap) {
for (const auto &E : ExtMap) {
std::string Version = std::to_string(E.second.MajorVersion) + "." +
std::to_string(E.second.MinorVersion);
- PrintExtension(E.first, Version, DescMap[E.first].str());
+ PrintExtension(E.first, Version, DescMap[E.first]);
}
outs() << "\nExperimental extensions\n";
@@ -243,7 +244,7 @@ void llvm::riscvExtensionsHelp(StringMap<StringRef> DescMap) {
for (const auto &E : ExtMap) {
std::string Version = std::to_string(E.second.MajorVersion) + "." +
std::to_string(E.second.MinorVersion);
- PrintExtension(E.first, Version, DescMap["experimental-" + E.first].str());
+ PrintExtension(E.first, Version, DescMap["experimental-" + E.first]);
}
outs() << "\nUse -march to specify the target's extension.\n"
More information about the llvm-commits
mailing list