[PATCH] D75723: [X86] Enable intrinsics _BitScan*

[Craig Topper via Phabricator via cfe-commits]

craig.topper added inline comments.


================
Comment at: clang/lib/Headers/ia32intrin.h:456
+  __extension__({                                                              \
+    long long c = (long long)(b);                                              \
+    unsigned char d;                                                           \
----------------
Variable name in macro needs to start with 2 underscores to avoid conflicts with user variable names. I need to think about whether that’s enough to avoid problems.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75723