[PATCH] D59367: [Sema] Adjust address space of operands in remaining builtin operators
Anastasia Stulova via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Mar 19 09:52:05 PDT 2019
Anastasia added a comment.
In D59367#1434353 <https://reviews.llvm.org/D59367#1434353>, @rjmccall wrote:
> This patch LGTM.
>
> You might consider adding tests for weird cases like class types with conversion operators to reference types. To a certain extent that sort of thing is unimplementable because the space of address spaces is infinite, so we can't fall back on adding a candidate for every possible address space; but even if we don't plan on addressing it, we can test our current behavior to check that we don't do something unexpected (like crash or give a nonsensical diagnostic).
Thanks for the suggestion. I will look into this!
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D59367/new/
https://reviews.llvm.org/D59367
More information about the cfe-commits
mailing list