[PATCH] D38781: [X86] Add CLWB intrinsic. clang part

Simon Pilgrim via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 12 09:01:36 PDT 2017

RKSimon added inline comments.

Comment at: lib/Headers/clwbintrin.h:34
+static __inline__ void __DEFAULT_FN_ATTRS
+_mm_clwb(void const *__m) {
Worth adding the doxygen description? You can probably just copy+paste+modify the _mm_clflush documentation.

Comment at: test/CodeGen/builtin-clwb.c:4
+#include <immintrin.h>
+void test_mm_clwb(const void *__m) {
Match what is done in the other files: use -ffreestanding instead of defining __MM_MALLOC_H and include the "top header":<x86intrin.h>


More information about the cfe-commits mailing list