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 _______________________________________________ llvm-branch-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
