[llvm] [llvm-extract] support unnamed bbs. (PR #135140)
Allin Lee via llvm-commits
llvm-commits at lists.llvm.org
Thu Apr 10 19:25:07 PDT 2025
https://github.com/AllinLeeYL updated https://github.com/llvm/llvm-project/pull/135140
>From 14bf34029bd89bb3be2829f9b524f5548e4a2cac Mon Sep 17 00:00:00 2001
From: Yilin Li <allinleeme at gmail.com>
Date: Thu, 10 Apr 2025 13:48:02 +0800
Subject: [PATCH] [llvm-extract] support unnamed bbs.
---
llvm/tools/llvm-extract/llvm-extract.cpp | 23 +++++++++++++++++++++--
1 file changed, 21 insertions(+), 2 deletions(-)
diff --git a/llvm/tools/llvm-extract/llvm-extract.cpp b/llvm/tools/llvm-extract/llvm-extract.cpp
index 648060acb392c..f2d6553924e15 100644
--- a/llvm/tools/llvm-extract/llvm-extract.cpp
+++ b/llvm/tools/llvm-extract/llvm-extract.cpp
@@ -90,10 +90,13 @@ static cl::list<std::string> ExtractBlocks(
"Each pair will create a function.\n"
"If multiple basic blocks are specified in one pair,\n"
"the first block in the sequence should dominate the rest.\n"
+ "If an unnamed basic block is to be extracted,\n"
+ "'%' should be added before the basic block variable names.\n"
"eg:\n"
" --bb=f:bb1;bb2 will extract one function with both bb1 and bb2;\n"
" --bb=f:bb1 --bb=f:bb2 will extract two functions, one with bb1, one "
- "with bb2."),
+ "with bb2.\n"
+ " --bb=f:%1 will extract one function with basic block 1;"),
cl::value_desc("function:bb1[;bb2...]"), cl::cat(ExtractCat));
// ExtractAlias - The alias to extract from the module.
@@ -355,9 +358,25 @@ int main(int argc, char **argv) {
for (StringRef BBName : P.second) {
// The function has been materialized, so add its matching basic blocks
// to the block extractor list, or fail if a name is not found.
+#ifndef NDEBUG
auto Res = llvm::find_if(*P.first, [&](const BasicBlock &BB) {
- return BB.getName() == BBName;
+ return BB.getNameOrAsOperand() == BBName;
});
+#else
+ llvm::Function::iterator Res;
+ if (BBName.substr(0, 1) == "%") {
+ Res = llvm::find_if(*P.first, [&](const BasicBlock &BB) {
+ std::string tmpName;
+ raw_string_ostream OS(tmpName);
+ BB.printAsOperand(OS, false);
+ return OS.str() == BBName;
+ });
+ } else {
+ Res = llvm::find_if(*P.first, [&](const BasicBlock &BB) {
+ return BB.getName() == BBName;
+ });
+ }
+#endif
if (Res == P.first->end()) {
errs() << argv[0] << ": function " << P.first->getName()
<< " doesn't contain a basic block named '" << BBName
More information about the llvm-commits
mailing list