[coco.generic] Remove deprecated methods (#1722)
author박종현/동작제어Lab(SR)/Staff Engineer/삼성전자 <jh1302.park@samsung.com>
Tue, 2 Oct 2018 09:56:14 +0000 (18:56 +0900)
committerGitHub Enterprise <noreply-CODE@samsung.com>
Tue, 2 Oct 2018 09:56:14 +0000 (18:56 +0900)
This commit removes deprecated methods that was introduced for
Object-level data managment.

Signed-off-by: Jonghyun Park <jh1302.park@samsung.com>
contrib/coco/generic/include/coco/IR/Data.h
contrib/coco/generic/include/coco/IR/PlainWeightContext.h
contrib/coco/generic/src/IR/Data.cpp
contrib/coco/generic/src/IR/Data.test.cpp

index 9a05ae6..163d1d4 100644 (file)
@@ -16,11 +16,6 @@ struct Data
 {
   virtual ~Data() = default;
 
-  // @brief Allocate blob space of 'size' bytes
-  //
-  // WARN Deprecated
-  virtual coco::BlobID allocate(uint32_t size) = 0;
-
   virtual PlainWeightContext<float> *f32(void) = 0;
   virtual const PlainWeightContext<float> *f32(void) const = 0;
 
index 0354a26..1f8fb20 100644 (file)
@@ -21,9 +21,6 @@ template <typename T> struct PlainWeightContext
 
   virtual void allocate(const KernelObject *) = 0;
 
-  // @brief Link pre-allocated blob space (denoted by BlobID) with coco::KernelObject
-  virtual void link(const BlobID &, const KernelObject *) = 0;
-
   virtual std::unique_ptr<nncc::core::ADT::kernel::Accessor<T>> access(const KernelObject *) = 0;
   virtual std::unique_ptr<nncc::core::ADT::kernel::Reader<T>> read(const KernelObject *) const = 0;
 };
index c3b3671..ce57199 100644 (file)
@@ -116,8 +116,6 @@ struct DataImpl final : public coco::Data
   std::unique_ptr<BlobContext> _blob;
   std::unique_ptr<PlainWeightContextImpl<float>> _fp32;
 
-  coco::BlobID allocate(uint32_t size) override { return _blob->allocate(size); }
-
   coco::PlainWeightContext<float> *f32(void) override { return _fp32.get(); }
   const coco::PlainWeightContext<float> *f32(void) const override { return _fp32.get(); }
 };
index 98157d4..8382370 100644 (file)
@@ -18,20 +18,21 @@ TEST(IR_DATA, construct)
 
 TEST(IR_DATA, allocate_and_link_kernel)
 {
+  using nncc::core::ADT::kernel::num_elements;
+
   // Create a kernel object
   auto module = coco::Module::create();
 
   const nncc::core::ADT::kernel::Shape shape{1, 1, 3, 3};
+  auto bag = module->entity()->bag()->create(9);
   auto obj = module->entity()->object()->create(shape);
 
-  auto data = coco::Data::create();
+  obj->bag(bag);
 
-  using nncc::core::ADT::kernel::num_elements;
+  auto data = coco::Data::create();
 
   // Create a blob and link it with kernel object
-  auto blob = data->allocate(num_elements(shape) * sizeof(float));
-  data->f32()->link(blob, obj);
-
+  data->f32()->allocate(obj);
   auto reader = data->f32()->read(obj);
   auto accessor = data->f32()->access(obj);