[PATCH] D14748: bmiintrin.h: Allow using the tzcnt intrinsics for non-BMI targets

Nico Weber via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 17 10:42:19 PST 2015


thakis added a comment.

Looks good. The reasoning is that without this, projects will put in an `#if __clang__ no tzcnt` path (e.g. http://ffmpeg.org/pipermail/ffmpeg-devel/2015-November/183408.html), and with this patch they won't and in a few years when people use build flags that allow BMI instructions, we'll magically start doing the fast thing.

(This is Eric's thing, but he sounded fine with this approach on IRC yesterday.)


http://reviews.llvm.org/D14748





More information about the cfe-commits mailing list