piiswrong commented on issue #7854: Basic CPU Kernel OMP selection based upon 
whether GPU has been used
URL: https://github.com/apache/incubator-mxnet/pull/7854#issuecomment-331691638
 
 
   This PR looks fine now and if it helps some people we should merge it.
   
   But tests are failing with compiler out of heap space.
   
   It feels like the sparse op PR is not very friendly to compilers.
   
   A simple fix I can think of is to split some of the operator cc files into 
multiple files, like what was done for elemwise_unary_basic and _extend. We are 
putting too many ops into the same file.
 
----------------------------------------------------------------
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