ebevhan added a comment.

In D58236#1467151 <https://reviews.llvm.org/D58236#1467151>, @Anastasia wrote:

> In D58236#1466263 <https://reviews.llvm.org/D58236#1466263>, @ebevhan wrote:
>
> > Well, it doesn't seem to me like there is consensus on prohibiting nested 
> > address space conversion like this.
> >
> > I can simply redo the patch to only include the bugfix on implicit 
> > conversions and drop the nesting level checks.
>
>
> I thought the conclusion is:
>
> - Explicit conversions allowed for nested pointers with a warning.
> - Implicit conversions are disallowed for nested pointers with an error.
>
>   Do you not see it this way?


Alright, that seems reasonable to me. Some of the patch goes away, then. I'll 
update it.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D58236/new/

https://reviews.llvm.org/D58236



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to