[PATCH] D21373: [Clang][bmi][intrinsics] Adding _mm_tzcnt_64 _mm_tzcnt_32 intrinsics to clang.

Simon Pilgrim via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 15 07:09:31 PDT 2016


RKSimon added a subscriber: RKSimon.
RKSimon added a reviewer: RKSimon.

================
Comment at: lib/Headers/bmiintrin.h:296
@@ -290,1 +295,3 @@
+}
+
 #ifdef __x86_64__
----------------
Why not just #define to __tzcnt_u32 like the (many) other duplicate tzcnt intrinsics we have:

```
#define _mm_tzcnt_32(a)     (__tzcnt_u32((a)))
```

Same for _mm_tzcnt_64

Also, please can you copy/paste/edit the doxygen comment so that its properly documented?


http://reviews.llvm.org/D21373





More information about the cfe-commits mailing list