tra added a comment.

Talked to @ahatanak over IRC. It appears that this patch may have exposed a 
preexisting bug.
Apparently `delete t;` in test/SemaCUDA/call-host-fn-from-device.cu does 
actually end up calling `__host__ operator delete`.  It should've picked 
`__device__ operator delete`, but it does not, so reporting an error here 
appears to be correct.

It's visible in AST and the IR.

@rsmith -- the original change was done a while back in 
https://reviews.llvm.org/rL283830.  I assume it worked at that time and wonder 
if it's a (possibly not-so-)recent regression.


Repository:
  rC Clang

https://reviews.llvm.org/D47757



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

Reply via email to