[incubator-mxnet] branch master updated: Additional fix for vector access. (#17230)

2020-02-16 Thread wkcn
This is an automated email from the ASF dual-hosted git repository.

wkcn pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git


The following commit(s) were added to refs/heads/master by this push:
 new 7743fb0  Additional fix for vector access.  (#17230)
7743fb0 is described below

commit 7743fb0c516032c39cbf15a7715a9d36a5cd8e18
Author: aws-taylor <57725958+aws-tay...@users.noreply.github.com>
AuthorDate: Sun Feb 16 07:57:46 2020 -0800

Additional fix for vector access.  (#17230)

* Additional fix for vector access. See 
https://github.com/apache/incubator-mxnet/commit/9634786f96388004f68c223d72e120ad425c2f12
 for the original.

* CI

* ci

* ci

* retrigger CI

* ci

Co-authored-by: JackieWu 
---
 3rdparty/mshadow/mshadow/dot_engine-inl.h | 9 +++--
 1 file changed, 3 insertions(+), 6 deletions(-)

diff --git a/3rdparty/mshadow/mshadow/dot_engine-inl.h 
b/3rdparty/mshadow/mshadow/dot_engine-inl.h
index 1a02eb9..d9abf29 100644
--- a/3rdparty/mshadow/mshadow/dot_engine-inl.h
+++ b/3rdparty/mshadow/mshadow/dot_engine-inl.h
@@ -421,12 +421,9 @@ struct BLASEngine {
   CBLAS_TRANSPOSE p_transa[GROUP_SIZE] = {cblas_a_trans};
   CBLAS_TRANSPOSE p_transb[GROUP_SIZE] = {cblas_b_trans};
 
-  std::vector pp_A;
-  std::vector pp_B;
-  std::vector pp_C;
-  pp_A.reserve(batch_count);
-  pp_B.reserve(batch_count);
-  pp_C.reserve(batch_count);
+  std::vector pp_A(batch_count, nullptr);
+  std::vector pp_B(batch_count, nullptr);
+  std::vector pp_C(batch_count, nullptr);
 
   auto m_k = m * k;
   auto k_n = k * n;



[incubator-mxnet] branch master updated: Additional fix for vector access. (#17230)

2020-02-16 Thread wkcn
This is an automated email from the ASF dual-hosted git repository.

wkcn pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git


The following commit(s) were added to refs/heads/master by this push:
 new 7743fb0  Additional fix for vector access.  (#17230)
7743fb0 is described below

commit 7743fb0c516032c39cbf15a7715a9d36a5cd8e18
Author: aws-taylor <57725958+aws-tay...@users.noreply.github.com>
AuthorDate: Sun Feb 16 07:57:46 2020 -0800

Additional fix for vector access.  (#17230)

* Additional fix for vector access. See 
https://github.com/apache/incubator-mxnet/commit/9634786f96388004f68c223d72e120ad425c2f12
 for the original.

* CI

* ci

* ci

* retrigger CI

* ci

Co-authored-by: JackieWu 
---
 3rdparty/mshadow/mshadow/dot_engine-inl.h | 9 +++--
 1 file changed, 3 insertions(+), 6 deletions(-)

diff --git a/3rdparty/mshadow/mshadow/dot_engine-inl.h 
b/3rdparty/mshadow/mshadow/dot_engine-inl.h
index 1a02eb9..d9abf29 100644
--- a/3rdparty/mshadow/mshadow/dot_engine-inl.h
+++ b/3rdparty/mshadow/mshadow/dot_engine-inl.h
@@ -421,12 +421,9 @@ struct BLASEngine {
   CBLAS_TRANSPOSE p_transa[GROUP_SIZE] = {cblas_a_trans};
   CBLAS_TRANSPOSE p_transb[GROUP_SIZE] = {cblas_b_trans};
 
-  std::vector pp_A;
-  std::vector pp_B;
-  std::vector pp_C;
-  pp_A.reserve(batch_count);
-  pp_B.reserve(batch_count);
-  pp_C.reserve(batch_count);
+  std::vector pp_A(batch_count, nullptr);
+  std::vector pp_B(batch_count, nullptr);
+  std::vector pp_C(batch_count, nullptr);
 
   auto m_k = m * k;
   auto k_n = k * n;