[llvm-branch-commits] [ThinLTO] Mark AssignGUIDPass as RequiredPassInfoMixin (PR #211987)

[via llvm-branch-commits]

llvmorg-github-actions[bot] wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-llvm-transforms

Author: Aiden Grossman (boomanaiden154)

<details>
<summary>Changes</summary>

PassInfoMixin is going away, so switch to RequiredPassInfoMixin and
remove the explicit isRequired implementation.


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


1 Files Affected:

- (modified) llvm/include/llvm/Transforms/Utils/AssignGUID.h (+1-3) 


``````````diff
diff --git a/llvm/include/llvm/Transforms/Utils/AssignGUID.h b/llvm/include/llvm/Transforms/Utils/AssignGUID.h
index 0622768f5df18..c15116417a475 100644
--- a/llvm/include/llvm/Transforms/Utils/AssignGUID.h
+++ b/llvm/include/llvm/Transforms/Utils/AssignGUID.h
@@ -25,7 +25,7 @@
 
 namespace llvm {
 
-class AssignGUIDPass : public PassInfoMixin<AssignGUIDPass> {
+class AssignGUIDPass : public RequiredPassInfoMixin<AssignGUIDPass> {
 public:
   AssignGUIDPass() = default;
 
@@ -36,8 +36,6 @@ class AssignGUIDPass : public PassInfoMixin<AssignGUIDPass> {
     return PreservedAnalyses::all();
   }
 
-  static bool isRequired() { return true; }
-
   // Let GlobalMerge assign a GUID for merged GVs, instead of needing to
   // traverse all the module; or instead of making GlobalValue::assignGUID
   // public.

``````````

</details>


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