[PATCH] D27627: Allow target to specify default address space for codegen

Yaxun Liu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Dec 13 14:09:27 PST 2016


yaxunl added a comment.

In https://reviews.llvm.org/D27627#621473, @rjmccall wrote:

> In https://reviews.llvm.org/D27627#619928, @yaxunl wrote:
>
> > > Because, notably, if you do that, then an attempt to pass &x as an int* will fail, which means this isn't really C++ anymore... and yet that appears to be exactly what you want.
> >
> > How about when generating alloca instruction, I insert addrspacecast to the default address space, then everything is in default address space.
>
>
> My question is really about your language design.  You have multiple address spaces in the implementation, but you're (apparently?) pretending that they're all part of the same address space in the source language.  How is that expected to work?  Does the default address space embed all other address spaces?


Yes the default address space embeds all other address spaces.


https://reviews.llvm.org/D27627





More information about the cfe-commits mailing list