[PATCH] D24571: [CUDA] Disallow overloading destructors.

[Reid Kleckner via cfe-commits]

rnk accepted this revision.
rnk added a reviewer: rnk.
rnk added a comment.
This revision is now accepted and ready to land.

lgtm



> SemaOverload.cpp:1131
>  
>    if (getLangOpts().CUDA && ConsiderCudaAttrs) {
>      CUDAFunctionTarget NewTarget = IdentifyCUDATarget(New),

I feel like we should exit early on destructors here, before we do any target checks. The assert also feels kind of trivial because we only come into this overload machinery if looking up New's DeclarationName found Old.

https://reviews.llvm.org/D24571