[llvm-bolt] Modify getLibPath to enable --runtime-instrumentation-lib option handle full path correctly.(llvm#99772)

[lltsdyp]

No description provided.

[github-actions]

Thank you for submitting a Pull Request (PR) to the LLVM Project!

This PR will be automatically labeled and the relevant teams will be
notified.

If you wish to, you can add reviewers by using the "Reviewers" section on this page.

If this is not working for you, it is probably because you do not have write
permissions for the repository. In which case you can instead tag reviewers by
name in a comment by using @ followed by their GitHub username.

If you have received no comments on your PR for a week, you can request a review
by "ping"ing the PR by adding a comment “Ping”. The common courtesy "ping" rate
is once a week. Please remember that you are asking for valuable time from other developers.

If you have further questions, they may be answered by the LLVM GitHub User Guide.

You can also ask questions in a comment on this PR, on the LLVM Discord or on the forums.

[llvmbot]

@llvm/pr-subscribers-bolt

Author: Zimo Ji (lltsdyp)

Changes

---

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

1 Files Affected:

• (modified) bolt/lib/RuntimeLibs/RuntimeLibrary.cpp (+6)

diff --git a/bolt/lib/RuntimeLibs/RuntimeLibrary.cpp b/bolt/lib/RuntimeLibs/RuntimeLibrary.cpp
index 276b034d71f96..0826847ee1189 100644
--- a/bolt/lib/RuntimeLibs/RuntimeLibrary.cpp
+++ b/bolt/lib/RuntimeLibs/RuntimeLibrary.cpp
@@ -28,6 +28,12 @@ void RuntimeLibrary::anchor() {}
 
 std::string RuntimeLibrary::getLibPath(StringRef ToolPath,
                                        StringRef LibFileName) {
+  // Handle full path.
+  // It is weird that append LibFileName to LibPath when user gives a full path.
+  if(LibFileName[0]=='/')
+  {
+    return LibFileName.str();
+  }
   StringRef Dir = llvm::sys::path::parent_path(ToolPath);
   SmallString<128> LibPath = llvm::sys::path::parent_path(Dir);
   llvm::sys::path::append(LibPath, "lib" LLVM_LIBDIR_SUFFIX);

[lltsdyp]

Hmm, I forgot to verify the existence of the runtime instrumentation library so I modified it and open a new PR

[lltsdyp]

Oops, it seems like my new commits were included in this PR so I don't need to open a new PR
Sorry for that, I am a noob :)

[rafaelauler]

Thanks for the fix! Currently #97130 PR is already working on this function, though. If that PR doesn't land, we can land this one.

[peterwaller-arm]

@lltsdyp the above PR now landed; looks like there are straightforward conflicts to resolve.