[PATCH][X86] __builtin_ctz/clz sometimed defined for zero input

Robinson, Paul Paul_Robinson at playstation.sony.com
Mon Oct 27 18:16:32 PDT 2014


I don't know about Mac, but on my Ubuntu, Intrin.h shows up in the expected place:

~/projects/upstream/writeable/obj$ find . -name Intrin.h
./Debug+Asserts/lib/clang/3.6.0/include/Intrin.h
./Debug+Asserts/lib/clang/3.5.0/include/Intrin.h
./Debug+Asserts/lib/clang/3.4/include/Intrin.h
./Debug+Asserts/lib/clang/3.5/include/Intrin.h

From: Chandler Carruth [mailto:chandlerc at google.com]
Sent: Monday, October 27, 2014 5:17 PM
To: Robinson, Paul
Cc: Sean Silva; cfe-commits at cs.uiuc.edu
Subject: Re: [PATCH][X86] __builtin_ctz/clz sometimed defined for zero input


On Mon, Oct 27, 2014 at 5:02 PM, Robinson, Paul <Paul_Robinson at playstation.sony.com<mailto:Paul_Robinson at playstation.sony.com>> wrote:
That makes lzcntintrin.h completely pointless.  They'd just be alternate names for the functions in Intrin.h.

If those names need to be supported for compatibility and portability then it still seems to have a point.

Note that Intrin.h doesn't exist on Linux or Mac.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20141028/b37edac2/attachment.html>


More information about the cfe-commits mailing list