jlebar added a comment.

In https://reviews.llvm.org/D25640#579238, @tra wrote:

> I'm OK with the change, but the comments suggest that things may be more 
> complicated.
>  How about disabling assert for CUDA only?


I don't think that is the right approach.  This function has nothing to do with 
CUDA, so either the assertion is valid and we shouldn't be calling it this way, 
or it's not valid and we should remove the assertion.

Richard, would appreciate some help with this one.


https://reviews.llvm.org/D25640



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

Reply via email to