[llvm] [InferAddressSpaces] apply InferAddressSpaces to inttoptr-zext-ptrtoint address expression. (PR #79108)

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 19 05:25:41 PDT 2024


arsenm wrote:

> I was wondering what I need to do to get it merged? Or do I need change something?

I'm not convinced this is safe. Why do you need this pattern handled? inttoptr/ptrtoint destroys pointer provenance and the general guideline is to never touch them. We only special case the case we do now because the IR is missing a way to bitcast between pointers with the same size. Is there some rationale this is safe wrt provenance? 

https://github.com/llvm/llvm-project/pull/79108


More information about the llvm-commits mailing list