arichardson marked an inline comment as done.
arichardson added inline comments.
================
Comment at: clang/lib/CodeGen/CGBuiltin.cpp:14322
+ if (SrcForMask->getType()->isPointerTy()) {
+ /// TODO: Use ptrmask instead of ptrtoint/inttoptr
+ // Result = Builder.CreateIntrinsic(
----------------
lebedev.ri wrote:
> Until we are there, can we still emit sane IR, without `inttoptr`?
> https://godbolt.org/z/atBtSx
It seems like avoiding the select might be better?
https://godbolt.org/z/UdPjZk
I've done that in the current version.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D71499/new/
https://reviews.llvm.org/D71499
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
