RKSimon added a comment.

It seems like part of the need for this is because the _mm_cmp_ps style 
intrinsics are defined as macros (to get around the problem of trying to use an 
immediate as an argument):

  #define _mm_cmp_ps(a, b, c) __extension__ ({ \
    (__m128)__builtin_ia32_cmpps((__v4sf)(__m128)(a), \
                                 (__v4sf)(__m128)(b), (c)); })

which means clang can't use a __target__("avx") attribute to stop their use.

Given that I'm happy with this patch's approach - anyone else have any 
suggestions?


http://reviews.llvm.org/D21306



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to