[llvm] [NFC] Remove isRequired() from analyses (PR #196121)

[via llvm-commits]

llvmorg-github-actions[bot] wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-llvm-ir

Author: Arthur Eubanks (aeubanks)

<details>
<summary>Changes</summary>

isRequired() isn't relevant for analyses.

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


2 Files Affected:

- (modified) llvm/include/llvm/CodeGen/AssignmentTrackingAnalysis.h (-2) 
- (modified) llvm/include/llvm/IR/Verifier.h (-1) 


``````````diff
diff --git a/llvm/include/llvm/CodeGen/AssignmentTrackingAnalysis.h b/llvm/include/llvm/CodeGen/AssignmentTrackingAnalysis.h
index 4e9f3af2111a7..2a72b28a6631c 100644
--- a/llvm/include/llvm/CodeGen/AssignmentTrackingAnalysis.h
+++ b/llvm/include/llvm/CodeGen/AssignmentTrackingAnalysis.h
@@ -131,8 +131,6 @@ class AssignmentTrackingAnalysis : public FunctionPass {
 
   bool runOnFunction(Function &F) override;
 
-  static bool isRequired() { return true; }
-
   void getAnalysisUsage(AnalysisUsage &AU) const override {
     AU.setPreservesAll();
   }
diff --git a/llvm/include/llvm/IR/Verifier.h b/llvm/include/llvm/IR/Verifier.h
index b0ab1db1b2b60..80e9b9284c9b7 100644
--- a/llvm/include/llvm/IR/Verifier.h
+++ b/llvm/include/llvm/IR/Verifier.h
@@ -118,7 +118,6 @@ class VerifierAnalysis : public AnalysisInfoMixin<VerifierAnalysis> {
 
   LLVM_ABI Result run(Module &M, ModuleAnalysisManager &);
   LLVM_ABI Result run(Function &F, FunctionAnalysisManager &);
-  static bool isRequired() { return true; }
 };
 
 /// Create a verifier pass.

``````````

</details>


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