[llvm] [SandboxIR][NFC] Move sandboxir::Use into a separate file (PR #99074)
via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 16 11:29:24 PDT 2024
https://github.com/vporpo created https://github.com/llvm/llvm-project/pull/99074
This helps avoid circular dependencies in a follow-up patch.
>From f3229a5e29bf3d6ec7b861679362651b19e22831 Mon Sep 17 00:00:00 2001
From: Vasileios Porpodas <vporpodas at google.com>
Date: Fri, 12 Jul 2024 10:35:30 -0700
Subject: [PATCH] [SandboxIR][NFC] Move sandboxir::Use into a separate file
This helps avoid circular dependencies in a follow-up patch.
---
llvm/include/llvm/SandboxIR/SandboxIR.h | 37 +-----------------
llvm/include/llvm/SandboxIR/Use.h | 52 +++++++++++++++++++++++++
2 files changed, 53 insertions(+), 36 deletions(-)
create mode 100644 llvm/include/llvm/SandboxIR/Use.h
diff --git a/llvm/include/llvm/SandboxIR/SandboxIR.h b/llvm/include/llvm/SandboxIR/SandboxIR.h
index d4ab8c3548a3c..d9e2e93b0c42b 100644
--- a/llvm/include/llvm/SandboxIR/SandboxIR.h
+++ b/llvm/include/llvm/SandboxIR/SandboxIR.h
@@ -61,6 +61,7 @@
#include "llvm/IR/Function.h"
#include "llvm/IR/User.h"
#include "llvm/IR/Value.h"
+#include "llvm/SandboxIR/Use.h"
#include "llvm/Support/raw_ostream.h"
#include <iterator>
@@ -75,42 +76,6 @@ class Instruction;
class User;
class Value;
-/// Represents a Def-use/Use-def edge in SandboxIR.
-/// NOTE: Unlike llvm::Use, this is not an integral part of the use-def chains.
-/// It is also not uniqued and is currently passed by value, so you can have
-/// more than one sandboxir::Use objects for the same use-def edge.
-class Use {
- llvm::Use *LLVMUse;
- User *Usr;
- Context *Ctx;
-
- /// Don't allow the user to create a sandboxir::Use directly.
- Use(llvm::Use *LLVMUse, User *Usr, Context &Ctx)
- : LLVMUse(LLVMUse), Usr(Usr), Ctx(&Ctx) {}
- Use() : LLVMUse(nullptr), Ctx(nullptr) {}
-
- friend class Value; // For constructor
- friend class User; // For constructor
- friend class OperandUseIterator; // For constructor
- friend class UserUseIterator; // For accessing members
-
-public:
- operator Value *() const { return get(); }
- Value *get() const;
- class User *getUser() const { return Usr; }
- unsigned getOperandNo() const;
- Context *getContext() const { return Ctx; }
- bool operator==(const Use &Other) const {
- assert(Ctx == Other.Ctx && "Contexts differ!");
- return LLVMUse == Other.LLVMUse && Usr == Other.Usr;
- }
- bool operator!=(const Use &Other) const { return !(*this == Other); }
-#ifndef NDEBUG
- void dump(raw_ostream &OS) const;
- void dump() const;
-#endif // NDEBUG
-};
-
/// Returns the operand edge when dereferenced.
class OperandUseIterator {
sandboxir::Use Use;
diff --git a/llvm/include/llvm/SandboxIR/Use.h b/llvm/include/llvm/SandboxIR/Use.h
new file mode 100644
index 0000000000000..a1b48ac3b966a
--- /dev/null
+++ b/llvm/include/llvm/SandboxIR/Use.h
@@ -0,0 +1,52 @@
+#ifndef LLVM_TRANSFORMS_SANDBOXIR_USE_H
+#define LLVM_TRANSFORMS_SANDBOXIR_USE_H
+
+#include "llvm/IR/Use.h"
+#include "llvm/Support/raw_ostream.h"
+
+namespace llvm {
+namespace sandboxir {
+
+class Context;
+class Value;
+class User;
+
+/// Represents a Def-use/Use-def edge in SandboxIR.
+/// NOTE: Unlike llvm::Use, this is not an integral part of the use-def chains.
+/// It is also not uniqued and is currently passed by value, so you can have
+/// more than one sandboxir::Use objects for the same use-def edge.
+class Use {
+ llvm::Use *LLVMUse;
+ User *Usr;
+ Context *Ctx;
+
+ /// Don't allow the user to create a sandboxir::Use directly.
+ Use(llvm::Use *LLVMUse, User *Usr, Context &Ctx)
+ : LLVMUse(LLVMUse), Usr(Usr), Ctx(&Ctx) {}
+ Use() : LLVMUse(nullptr), Ctx(nullptr) {}
+
+ friend class Value; // For constructor
+ friend class User; // For constructor
+ friend class OperandUseIterator; // For constructor
+ friend class UserUseIterator; // For accessing members
+
+public:
+ operator Value *() const { return get(); }
+ Value *get() const;
+ class User *getUser() const { return Usr; }
+ unsigned getOperandNo() const;
+ Context *getContext() const { return Ctx; }
+ bool operator==(const Use &Other) const {
+ assert(Ctx == Other.Ctx && "Contexts differ!");
+ return LLVMUse == Other.LLVMUse && Usr == Other.Usr;
+ }
+ bool operator!=(const Use &Other) const { return !(*this == Other); }
+#ifndef NDEBUG
+ void dump(raw_ostream &OS) const;
+ void dump() const;
+#endif // NDEBUG
+};
+} // namespace sandboxir
+} // namespace llvm
+
+#endif // LLVM_TRANSFORMS_SANDBOXIR_USE_H
More information about the llvm-commits
mailing list