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

hfinkel at anl.gov hfinkel at anl.gov
Sat Jul 25 19:40:33 PDT 2015

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?


More information about the llvm-commits mailing list