[PATCH] D62574: Initial draft of target-configurable address spaces.
Danila Malyutin via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Jul 6 12:54:11 PDT 2020
danilaml added a comment.
What are the remaining roadblocks left before this patch can be merged? I'm interested in having a target-specific way to define the allowed explicit/implicit address space conversions.
Also, it appears that currently whether implicit casts between pointers of different AS are allowed is determined by the superset relations only, however https://reviews.llvm.org/D73360 introduced another (openCL-specific) variable into the mix: whether the conversion is a top level or not. IMHO, this should also be configured by the target, although I'm not sure whether it needs a separate hook (isBitcastNoop or similar?) or it needs to check the legality of the implicit casts differently.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D62574/new/
https://reviews.llvm.org/D62574
More information about the cfe-commits
mailing list