[llvm] [Orc][LibResolver] Refactor resolver internals and simplify symbol resolution. (PR #169161)
via llvm-commits
llvm-commits at lists.llvm.org
Thu Dec 4 02:17:47 PST 2025
github-actions[bot] wrote:
<!--LLVM CODE FORMAT COMMENT: {clang-format}-->
:warning: C/C++ code formatter, clang-format found issues in your code. :warning:
<details>
<summary>
You can test this locally with the following command:
</summary>
``````````bash
git-clang-format --diff origin/main HEAD --extensions h,cpp -- llvm/include/llvm/ExecutionEngine/Orc/TargetProcess/LibraryResolver.h llvm/include/llvm/ExecutionEngine/Orc/TargetProcess/LibraryScanner.h llvm/lib/ExecutionEngine/Orc/TargetProcess/LibraryResolver.cpp llvm/lib/ExecutionEngine/Orc/TargetProcess/LibraryScanner.cpp llvm/unittests/ExecutionEngine/Orc/LibraryResolverTest.cpp --diff_from_common_commit
``````````
:warning:
The reproduction instructions above might return results for more than one PR
in a stack if you are using a stacked PR workflow. You can limit the results by
changing `origin/main` to the base branch/commit you want to compare against.
:warning:
</details>
<details>
<summary>
View the diff from clang-format here.
</summary>
``````````diff
diff --git a/llvm/lib/ExecutionEngine/Orc/TargetProcess/LibraryScanner.cpp b/llvm/lib/ExecutionEngine/Orc/TargetProcess/LibraryScanner.cpp
index 025da92d0..e74568e6b 100644
--- a/llvm/lib/ExecutionEngine/Orc/TargetProcess/LibraryScanner.cpp
+++ b/llvm/lib/ExecutionEngine/Orc/TargetProcess/LibraryScanner.cpp
@@ -65,15 +65,14 @@ static std::string getHostFileFormatName() {
}
bool ObjectFileLoader::isArchitectureCompatible(const object::ObjectFile &Obj) {
- return Obj.getFileFormatName() == getHostFileFormatName();
+ return Obj.getFileFormatName() == getHostFileFormatName();
}
Expected<object::OwningBinary<object::ObjectFile>>
ObjectFileLoader::loadObjectFileWithOwnership(StringRef FilePath) {
LLVM_DEBUG(dbgs() << "ObjectFileLoader: Attempting to open file " << FilePath
<< "\n";);
- if (auto ObjOrErr =
- object::ObjectFile::createObjectFile(FilePath)) {
+ if (auto ObjOrErr = object::ObjectFile::createObjectFile(FilePath)) {
LLVM_DEBUG(dbgs() << "ObjectFileLoader: Detected object file\n";);
@@ -157,8 +156,8 @@ ObjectFileLoader::loadObjectFileWithOwnership(StringRef FilePath) {
}
return createStringError(inconvertibleErrorCode(),
- "Not a compatible object file : %s",
- FilePath.str().c_str());
+ "Not a compatible object file : %s",
+ FilePath.str().c_str());
}
template <class ELFT>
``````````
</details>
https://github.com/llvm/llvm-project/pull/169161
More information about the llvm-commits
mailing list