Skip to content

Conversation

bogner
Copy link
Contributor

@bogner bogner commented Aug 14, 2024

This analysis can't be used with other analyses if this isn't set.

Created using spr 1.3.5-bogner
@llvmbot
Copy link
Member

llvmbot commented Aug 14, 2024

@llvm/pr-subscribers-backend-directx

Author: Justin Bogner (bogner)

Changes

This analysis can't be used with other analyses if this isn't set.


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

1 Files Affected:

  • (modified) llvm/lib/Target/DirectX/DXILResourceAnalysis.h (+4)
diff --git a/llvm/lib/Target/DirectX/DXILResourceAnalysis.h b/llvm/lib/Target/DirectX/DXILResourceAnalysis.h
index 3a2b8a9fd39d59..26d9237d51b49c 100644
--- a/llvm/lib/Target/DirectX/DXILResourceAnalysis.h
+++ b/llvm/lib/Target/DirectX/DXILResourceAnalysis.h
@@ -57,6 +57,10 @@ class DXILResourceMDWrapper : public ModulePass {
   /// Calculate the DXILResource for the module.
   bool runOnModule(Module &M) override;
 
+  void getAnalysisUsage(AnalysisUsage &AU) const override {
+    AU.setPreservesAll();
+  }
+
   void print(raw_ostream &O, const Module *M = nullptr) const override;
 };
 } // namespace llvm

bogner added a commit to bogner/llvm-project that referenced this pull request Aug 14, 2024
This analysis can't be used with other analyses if this isn't set.

Pull Request: llvm#104244
Copy link
Member

@hekota hekota left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@bogner bogner merged commit f999b32 into main Aug 16, 2024
@bogner bogner deleted the users/bogner/sprdirectx-add-missing-analysis-usage-to-dxilresourcemdwrapper branch August 16, 2024 18:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants