[llvm] [BOLT] Fix setHasSymbolsWithFileName (PR #92625)

via llvm-commits llvm-commits at lists.llvm.org
Fri May 17 18:37:48 PDT 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-bolt

Author: Amir Ayupov (aaupov)

<details>
<summary>Changes</summary>

The function is used to ignore the parameter and set
`HasSymbolsWithFileName` unconditionally.


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


1 Files Affected:

- (modified) bolt/include/bolt/Core/BinaryContext.h (+1-1) 


``````````diff
diff --git a/bolt/include/bolt/Core/BinaryContext.h b/bolt/include/bolt/Core/BinaryContext.h
index 75765819ac464..792c09dd7c467 100644
--- a/bolt/include/bolt/Core/BinaryContext.h
+++ b/bolt/include/bolt/Core/BinaryContext.h
@@ -359,7 +359,7 @@ class BinaryContext {
   void setFileBuildID(StringRef ID) { FileBuildID = std::string(ID); }
 
   bool hasSymbolsWithFileName() const { return HasSymbolsWithFileName; }
-  void setHasSymbolsWithFileName(bool Value) { HasSymbolsWithFileName = true; }
+  void setHasSymbolsWithFileName(bool Value) { HasSymbolsWithFileName = Value; }
 
   /// Return true if relocations against symbol with a given name
   /// must be created.

``````````

</details>


https://github.com/llvm/llvm-project/pull/92625


More information about the llvm-commits mailing list