[llvm] InferAddressSpaces: Handle llvm.fake.use (PR #109567)

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 24 05:49:00 PDT 2024


arsenm wrote:

> > I do not think this is a reasonable restriction to have.
> 
> That might be fine - I'm not the original author so I don't know the justification for the restriction when it was written, but it's likely simply that we never produce fake use intrinsics with more than one argument, and so code that handles it uses the hardcoded assumption that there is one argument; now that this intrinsic is part of upstream LLVM, allowing it to be used as a full variadic would be reasonable unless there are other technical constraints.

OK, I misread this. I was interpreting this as a single value cannot appear multiple times in the argument list. If the restriction is only one argument, that's a more reasonable restriction 

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


More information about the llvm-commits mailing list