steakhal added a comment. In D86295#2228745 <https://reviews.llvm.org/D86295#2228745>, @xazax.hun wrote:
> But unless you add a comment one will probably replace the offset with an > optional as the part of a refactoring. Sure, I will leave a note. I was thinking using a PointerIntUnion though. I haven't used it yet, but it's mught be a good candidate. What do you think? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D86295/new/ https://reviews.llvm.org/D86295 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits