NFC: Introduce new ValuePtr/ValueRef typedefs to simplify the transition to Value being value-typed.

This is an initial step to refactoring the representation of OpResult as proposed in: https://groups.google.com/a/tensorflow.org/g/mlir/c/XXzzKhqqF_0/m/v6bKb08WCgAJ

This change will make it much simpler to incrementally transition all of the existing code to use value-typed semantics.

PiperOrigin-RevId: 286844725

1 files changed, 4 insertions, 4 deletions

diff --git a/mlir/unittests/IR/OperationSupportTest.cpp b/mlir/unittests/IR/OperationSupportTest.cpp
index 80f82ac3e5d..d7dae4648fe 100644
--- a/mlir/unittests/IR/OperationSupportTest.cpp
+++ b/mlir/unittests/IR/OperationSupportTest.cpp
@@ -25,7 +25,7 @@ using namespace mlir::detail;
 
 namespace {
 Operation *createOp(MLIRContext *context, bool resizableOperands,
-                    ArrayRef<Value *> operands = llvm::None,
+                    ArrayRef<ValuePtr> operands = llvm::None,
                     ArrayRef<Type> resultTypes = llvm::None) {
   return Operation::create(
       UnknownLoc::get(context), OperationName("foo.bar", context), resultTypes,
@@ -39,7 +39,7 @@ TEST(OperandStorageTest, NonResizable) {
   Operation *useOp =
       createOp(&context, /*resizableOperands=*/false, /*operands=*/llvm::None,
                builder.getIntegerType(16));
-  Value *operand = useOp->getResult(0);
+  ValuePtr operand = useOp->getResult(0);
 
   // Create a non-resizable operation with one operand.
   Operation *user = createOp(&context, /*resizableOperands=*/false, operand,
@@ -68,7 +68,7 @@ TEST(OperandStorageDeathTest, AddToNonResizable) {
   Operation *useOp =
       createOp(&context, /*resizableOperands=*/false, /*operands=*/llvm::None,
                builder.getIntegerType(16));
-  Value *operand = useOp->getResult(0);
+  ValuePtr operand = useOp->getResult(0);
 
   // Create a non-resizable operation with one operand.
   Operation *user = createOp(&context, /*resizableOperands=*/false, operand,
@@ -88,7 +88,7 @@ TEST(OperandStorageTest, Resizable) {
   Operation *useOp =
       createOp(&context, /*resizableOperands=*/false, /*operands=*/llvm::None,
                builder.getIntegerType(16));
-  Value *operand = useOp->getResult(0);
+  ValuePtr operand = useOp->getResult(0);
 
   // Create a resizable operation with one operand.
   Operation *user = createOp(&context, /*resizableOperands=*/true, operand,