[PATCH] D50168: [Builtins] Implement __builtin_clrsb to be compatible with gcc

[Benjamin Kramer via Phabricator via cfe-commits]

bkramer added a comment.

Test case?



================
Comment at: lib/CodeGen/CGBuiltin.cpp:1563
+    Value *Result = Builder.CreateCall(F, {Tmp, Builder.getTrue()});
+    if (Result->getType() != ResultType)
+      Result = Builder.CreateIntCast(Result, ResultType, /*isSigned*/true,
----------------
CreateIntCast just does nothing if the types match, so this check isn't needed.


https://reviews.llvm.org/D50168



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits