[PATCH] D86229: [X86] Enable constexpr on POPCNT intrinsics (PR31446)

Simon Pilgrim via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 19 10:11:38 PDT 2020


RKSimon added inline comments.


================
Comment at: clang/lib/Headers/popcntintrin.h:16
 
+#ifdef __cplusplus
+#define __DEFAULT_FN_ATTRS_CONSTEXPR __DEFAULT_FN_ATTRS constexpr
----------------
craig.topper wrote:
> Do we need to check for C++11?
I don't actually know - its trivial to add a __cplusplus >= 201103L check but I don't know what we are expected to support in compiler headers like these.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D86229/new/

https://reviews.llvm.org/D86229



More information about the cfe-commits mailing list