ptrendx commented on issue #14067: Increase perfomance of BulkAppend and 
BulkFlush
URL: https://github.com/apache/incubator-mxnet/pull/14067#issuecomment-461145342
 
 
   @junrushao1994 2 reasons: 
    - std::move to lambda is not supported before C++14 without some arcane 
template tricks, which I did not want to put there
    - when I tried it (disregarding warnings from GCC that I'm using C++14 
feature), I got errors in some other places in the engine that I try to use 
deleted copy function of those lambdas and did not investigate further whether 
that's a bigger issue or just a lack of std::move in few places.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to