commit fed275d76d8322d51872845378adc0058c02bfc1
author Gian Marco Iodice <gianmarco.iodice@arm.com> Tue Aug 06 10:44:02 2019 +0100
committer Gian Marco Iodice <gianmarco.iodice@arm.com> Thu Aug 08 09:53:46 2019 +0000
tree ca9c3e9769273dbe303952876e973cd72ff963b6
parent 78a5d220f168f837efc087eeb1c4102a6e93a111

COMPMID-2546: Fix kernel name tag invalid ptr for NEGEMMAssemblyWrapperKernel.h

Change-Id: Iafc53602c9e8795072939dc14782c4bc49940812
Signed-off-by: Gian Marco Iodice <gianmarco.iodice@arm.com>
Reviewed-on: https://review.mlplatform.org/c/1700
Reviewed-by: Giorgio Arena <giorgio.arena@arm.com>
Tested-by: Arm Jenkins <bsgcomp@arm.com>

arm_compute/core/NEON/kernels/assembly/NEGEMMAssemblyWrapperKernel.h

diff --git a/arm_compute/core/NEON/kernels/assembly/NEGEMMAssemblyWrapperKernel.h b/arm_compute/core/NEON/kernels/assembly/NEGEMMAssemblyWrapperKernel.h
index 084c3f2..53ec3e1 100644
--- a/arm_compute/core/NEON/kernels/assembly/NEGEMMAssemblyWrapperKernel.h
+++ b/arm_compute/core/NEON/kernels/assembly/NEGEMMAssemblyWrapperKernel.h
@@ -53,7 +53,7 @@
     /** Constructor
      */
     NEGEMMAssemblyWrapperKernel()
-        : _kernel(nullptr), _kernel_name_tag()
+        : _kernel(nullptr), _name("NEGEMMAssemblyWrapperKernel")
     {
     }
 
@@ -63,12 +63,7 @@
 
     const char *name() const override
     {
-        std::string name = "NEGEMMAssemblyWrapperKernel";
-        if(!_kernel_name_tag.empty())
-        {
-            name += "/" + _kernel_name_tag;
-        }
-        return name.c_str();
+        return _name.c_str();
     }
     // Inherited methods overridden:
     void run(const Window &window, const ThreadInfo &info) override
@@ -88,16 +83,20 @@
     {
         ARM_COMPUTE_ERROR_ON_NULLPTR((reinterpret_cast<void *>(kernel)));
         _kernel          = kernel;
-        _kernel_name_tag = kernel_name_tag;
         auto   win_last  = _kernel->get_window_size();
         Window win;
         win.set(Window::DimX, Window::Dimension(0, win_last, 1));
         INEKernel::configure(win);
+
+        if(!kernel_name_tag.empty())
+        {
+            _name += "/" + kernel_name_tag;
+        }
     }
 
 private:
     arm_gemm::GemmCommon<TypeInput, TypeOutput> *_kernel;
-    std::string _kernel_name_tag;
+    std::string _name;
 };
 } // namespace arm_compute
 #endif /* __ARM_COMPUTE_ASSEMBLY_GEMM_KERNEL_WRAPPER_KERNEL_H__ */