[llvm-branch-commits] [PowerPC][AIX] Mark LowerCommentStringPass as RequiredPassInfoMixin (PR #212018)
via llvm-branch-commits
llvm-branch-commits at lists.llvm.org
Sat Jul 25 05:22:49 PDT 2026
llvmorg-github-actions[bot] wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-llvm-transforms
Author: Aiden Grossman (boomanaiden154)
<details>
<summary>Changes</summary>
Directly using isRequired is deprecated and PassInfoMixin will soon be
moved to a detail namespace.
---
Full diff: https://github.com/llvm/llvm-project/pull/212018.diff
1 Files Affected:
- (modified) llvm/include/llvm/Transforms/Utils/LowerCommentStringPass.h (+2-3)
``````````diff
diff --git a/llvm/include/llvm/Transforms/Utils/LowerCommentStringPass.h b/llvm/include/llvm/Transforms/Utils/LowerCommentStringPass.h
index e78b36b3314ac..f7402c738ca56 100644
--- a/llvm/include/llvm/Transforms/Utils/LowerCommentStringPass.h
+++ b/llvm/include/llvm/Transforms/Utils/LowerCommentStringPass.h
@@ -12,11 +12,10 @@
#include "llvm/IR/PassManager.h"
namespace llvm {
-class LowerCommentStringPass : public PassInfoMixin<LowerCommentStringPass> {
+class LowerCommentStringPass
+ : public RequiredPassInfoMixin<LowerCommentStringPass> {
public:
LLVM_ABI PreservedAnalyses run(Module &M, ModuleAnalysisManager &AM);
-
- static bool isRequired() { return true; }
};
} // namespace llvm
``````````
</details>
https://github.com/llvm/llvm-project/pull/212018
More information about the llvm-branch-commits
mailing list