larroy commented on issue #14192: [MXNET-1324] Add NaiveRunGraph to imperative 
utils
URL: https://github.com/apache/incubator-mxnet/pull/14192#issuecomment-501887455
 
 
   You are inlining a big function, this will cause code bloat at no benefit. 
Let the compiler do the inline for you in the majority of the cases. I removed 
the inline anyway so it's no longer in master. 
   
   See also: https://github.com/apache/incubator-mxnet/issues/15182

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


With regards,
Apache Git Services

Reply via email to