[PATCH] D35184: X86 Intrinsics: _bit_scan_forward should not be under #ifdef __RDRND__

2017-07-10 Thread Zvi Rackover via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL307524: X86 Intrinsics: _bit_scan_forward should not be under #ifdef __RDRND__ (authored by zvi). Repository: rL LLVM https://reviews.llvm.org/D35184 Files: cfe/trunk/lib/Headers/immintrin.h cfe/t

[PATCH] D35184: X86 Intrinsics: _bit_scan_forward should not be under #ifdef __RDRND__

2017-07-09 Thread Zvi Rackover via Phabricator via cfe-commits
zvi updated this revision to Diff 105788. zvi added a comment. rdrand64_step should be under the ifdef https://reviews.llvm.org/D35184 Files: lib/Headers/immintrin.h test/CodeGen/bitscan-builtins.c Index: test/CodeGen/bitscan-builtins.c

[PATCH] D35184: X86 Intrinsics: _bit_scan_forward should not be under #ifdef __RDRND__

2017-07-09 Thread Zvi Rackover via Phabricator via cfe-commits
zvi added inline comments. Comment at: lib/Headers/immintrin.h:230 static __inline__ int __attribute__((__always_inline__, __nodebug__, __target__("rdrnd"))) _rdrand64_step(unsigned long long *__p) { craig.topper wrote: > Looks like we now aren't removing rdr

[PATCH] D35184: X86 Intrinsics: _bit_scan_forward should not be under #ifdef __RDRND__

2017-07-09 Thread Zvi Rackover via Phabricator via cfe-commits
zvi created this revision. The _bit_scan_forward and _bit_scan_reverse intrinsics were accidentally masked under the preprocessor checks that prune intrinsics definitions for the benefit of faster compile-time on Windows. This patch moves the definitons out of that region. Fixes pr33722 https:/