[clang] [clang] Separate Intel ADC instrinsics from ADX intrinsics (PR #75992)

Phoebe Wang via cfe-commits cfe-commits at lists.llvm.org
Tue Dec 19 17:50:58 PST 2023


================
@@ -580,8 +580,7 @@ _storebe_i64(void * __P, long long __D) {
 #include <cetintrin.h>
 #endif
 
-/* Some intrinsics inside adxintrin.h are available only on processors with ADX,
- * whereas others are also available at all times. */
----------------
phoebewang wrote:

Sorry, just noticed this. Since we removed the comments, should we use the same format for `adxintrin.h` and add comment for `adcintrin.h`, e.g., 

```
/* Intrinsics inside adcintrin.h are available at all times. */
#include <adcintrin.h>

#if !(defined(_MSC_VER) || defined(__SCE__)) || __has_feature(modules) ||      \
    defined(__ADX__)
#include <adxintrin.h>
#endif
```

https://github.com/llvm/llvm-project/pull/75992


More information about the cfe-commits mailing list