[PATCH] D25264: Implement MS _BitScan intrinsics

[Albert Gutowski via cfe-commits]

agutowski added inline comments.


> CGBuiltin.cpp:2665
> +    BasicBlock *End = createBasicBlock("bitscan_end", this->CurFn);
> +    Builder.SetInsertPoint(End);
> +    PHINode *Result = Builder.CreatePHI(ResultType, 2, "bitscan_result");

Is this line needed? I took it from __builtin_fpclassify, but I don't know what could be its purpose (it's repeated below, where the "bitscan_end" block really starts).

https://reviews.llvm.org/D25264