[PATCH] D55677: [Builltins][X86] Provide implementations of __lzcnt16, __lzcnt, __lzcnt64 for MS compatibility. Remove declarations from intrin.h and implementations from lzcntintrin.h

Reid Kleckner via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 13 15:19:17 PST 2018


rnk added a comment.

This has the effect of removing __lzcnt* from non-MSVC targets using immintrin.h. Is that intended? It doesn't seem like these implementations were guarded by ifdef _MSC_VER.

Otherwise, yep, makes sense.


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

https://reviews.llvm.org/D55677





More information about the cfe-commits mailing list