pengfei added a comment. In D125164#3498752 <https://reviews.llvm.org/D125164#3498752>, @RKSimon wrote:
> Actually the ia32_tzcnt builtins should stay the way they are - other C/C++ > intrinsics return unsigned so we'd still end up with adding explicit casts No problem, adding explicit casts look good to me. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D125164/new/ https://reviews.llvm.org/D125164 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits