[GitHub] [incubator-mxnet] kpuatamazon commented on pull request #17559: [MXNET-1446] Quantization: intgemm matrix multiply wrappers

[GitBox <gi...@apache.org>]

kpuatamazon commented on pull request #17559:
URL: https://github.com/apache/incubator-mxnet/pull/17559#issuecomment-657550007


   The mac error turns out to be a clang bug in that target attributes don't fully work as well as the command line.  
   
   Here's `example.cc`:
   ```C++
   #include <immintrin.h>
   __attribute__ ((target ("avx2"))) void Foo(__m256i sum0) {
     asm("" : [sum0] "+x" (sum0));
   }
   ```
   
   It compiles fine with `-mavx2`:
   ```
   clang++-7 -mavx2  -O3 -c example.cc
   ```
   Then doesn't compile without `-mavx2`:
   ```
   clang++-7 -O3 -c example.cc
   example.cc:3:25: error: invalid output size for constraint '+x'
     asm("" : [sum0] "+x" (sum0));
                           ^
   1 error generated.
   ```
   Even though target attributes are supposed to act like they appeared on the command line.  Ugh, I'll get started on a workaround.  


----------------------------------------------------------------
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:
users@infra.apache.org