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

Craig Topper via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 13 15:21:38 PST 2018


craig.topper added a comment.

yeah I'm now realizing that gcc does implement __lzcnt16 and __lzcnt64. How do I make this work? I tried guarding them with _MSC_VER, but I got redefinition errors on a couple tests from that. Does -ms-extensions not define _MSC_VER?


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

https://reviews.llvm.org/D55677





More information about the cfe-commits mailing list