[PATCH] D11242: Fix assert when inlining a constantexpr addrspacecast

[hfinkel at anl.gov]

hfinkel added a subscriber: hfinkel.
hfinkel added a comment.

> Accumulating the offset through the addrspacecast doesn't make much sense


Why does it not make sense? It just means that the base object is in a different address space from the original derived pointer. The LangRef says:

> Note that if the address space conversion is legal then both result and operand refer to the same memory location.


and so, unless the addresscast were somehow illegal (is that possible for a constant expression addresscast?) then it should have a well-defined meaning. Am I missing something?


http://reviews.llvm.org/D11242