From: Lei Zhang <antiagainst@google.com>
Date: Wed, 24 Apr 2019 15:51:08 +0000 (-0700)
Subject:     Add MultiResultTraitBase
X-Git-Tag: llvmorg-11-init~1466^2~1890
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=9f934f2a59a68e44647ef7f3cd5b45878c32f38f;p=platform%2Fupstream%2Fllvm.git

Add MultiResultTraitBase

    Similar to MultiOperandTraitBase, this can simply the implementation of
    NResults, AtLeastNResults, and VariadicResults.

--

PiperOrigin-RevId: 245052333
---

diff --git a/mlir/include/mlir/IR/OpDefinition.h b/mlir/include/mlir/IR/OpDefinition.h
index f31ed11..a42ad5a 100644
--- a/mlir/include/mlir/IR/OpDefinition.h
+++ b/mlir/include/mlir/IR/OpDefinition.h
@@ -470,6 +470,37 @@ public:
   }
 };
 
+namespace detail {
+/// Utility trait base that provides accessors for derived traits that have
+/// multiple results.
+template <typename ConcreteType, template <typename> class TraitType>
+struct MultiResultTraitBase : public TraitBase<ConcreteType, TraitType> {
+  using result_iterator = Operation::result_iterator;
+  using result_range = Operation::result_range;
+
+  /// Return the number of results.
+  unsigned getNumResults() { return this->getOperation()->getNumResults(); }
+
+  /// Return the result at index 'i'.
+  Value *getResult(unsigned i) { return this->getOperation()->getResult(i); }
+
+  /// Set the result at index 'i' to 'value'.
+  void setResult(unsigned i, Value *value) {
+    this->getOperation()->setResult(i, value);
+  }
+
+  /// Return the type of the `i`-th result.
+  Type getType(unsigned i) { return getResult(i)->getType(); }
+
+  /// Result iterator access.
+  result_iterator result_begin() {
+    return this->getOperation()->result_begin();
+  }
+  result_iterator result_end() { return this->getOperation()->result_end(); }
+  result_range getResults() { return this->getOperation()->getResults(); }
+};
+} // end namespace detail
+
 /// This class provides return value APIs for ops that are known to have a
 /// single result.
 template <typename ConcreteType>
@@ -499,14 +530,9 @@ public:
 template <unsigned N> class NResults {
 public:
   template <typename ConcreteType>
-  class Impl : public TraitBase<ConcreteType, NResults<N>::Impl> {
+  class Impl
+      : public detail::MultiResultTraitBase<ConcreteType, NResults<N>::Impl> {
   public:
-    static unsigned getNumResults() { return N; }
-
-    Value *getResult(unsigned i) { return this->getOperation()->getResult(i); }
-
-    Type getType(unsigned i) { return getResult(i)->getType(); }
-
     static LogicalResult verifyTrait(Operation *op) {
       return impl::verifyNResults(op, N);
     }
@@ -521,12 +547,9 @@ public:
 template <unsigned N> class AtLeastNResults {
 public:
   template <typename ConcreteType>
-  class Impl : public TraitBase<ConcreteType, AtLeastNResults<N>::Impl> {
+  class Impl : public detail::MultiResultTraitBase<ConcreteType,
+                                                   AtLeastNResults<N>::Impl> {
   public:
-    Value *getResult(unsigned i) { return this->getOperation()->getResult(i); }
-
-    Type getType(unsigned i) { return getResult(i)->getType(); }
-
     static LogicalResult verifyTrait(Operation *op) {
       return impl::verifyAtLeastNResults(op, N);
     }
@@ -536,26 +559,8 @@ public:
 /// This class provides the API for ops which have an unknown number of
 /// results.
 template <typename ConcreteType>
-class VariadicResults : public TraitBase<ConcreteType, VariadicResults> {
-public:
-  unsigned getNumResults() { return this->getOperation()->getNumResults(); }
-
-  Value *getResult(unsigned i) { return this->getOperation()->getResult(i); }
-
-  void setResult(unsigned i, Value *value) {
-    this->getOperation()->setResult(i, value);
-  }
-
-  // Support result iteration.
-  using result_iterator = Operation::result_iterator;
-  result_iterator result_begin() {
-    return this->getOperation()->result_begin();
-  }
-  result_iterator result_end() { return this->getOperation()->result_end(); }
-  llvm::iterator_range<result_iterator> getResults() {
-    return this->getOperation()->getResults();
-  }
-};
+class VariadicResults
+    : public detail::MultiResultTraitBase<ConcreteType, VariadicResults> {};
 
 /// This class provides verification for ops that are known to have the same
 /// operand and result shape: both are scalars, vectors/tensors of the same
diff --git a/mlir/include/mlir/IR/Operation.h b/mlir/include/mlir/IR/Operation.h
index aacf9ee..b657953 100644
--- a/mlir/include/mlir/IR/Operation.h
+++ b/mlir/include/mlir/IR/Operation.h
@@ -203,9 +203,12 @@ public:
 
   // Support result iteration.
   using result_iterator = ResultIterator;
+  using result_range = llvm::iterator_range<result_iterator>;
+
   result_iterator result_begin();
   result_iterator result_end();
-  llvm::iterator_range<result_iterator> getResults();
+
+  result_range getResults();
 
   MutableArrayRef<OpResult> getOpResults() {
     return {getTrailingObjects<OpResult>(), numResults};