tra added a comment. In D111665#3059427 <https://reviews.llvm.org/D111665#3059427>, @jdoerfert wrote:
> Not loving the magic constants here but I don't think we have a enum or > similar right now. Yup. > I also have to question the people that choose `size_t` here... we will end > up with int2ptr(ptr2int(...)) IR everywhere if this is actually used (outside > the asm uses in cuda). I guess size_t was 'good enough' to accommodate all pointer sizes (though it should've been `uintptr_t`). I think this chain of conversions gets quickly instcombined away even at `-O1`: E.g: https://godbolt.org/z/4vd94cEsj Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111665/new/ https://reviews.llvm.org/D111665 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
