[clang] [MS][clang] Make sure vector deleting dtor calls correct operator delete (PR #133950)

[Devon Loehr via cfe-commits]

DKLoehr wrote:

I'm not familiar enough to say what we should be doing; perhaps @rnk or @zmodem have thoughts? It seems like we should at least emit a better error message, since just looking at the code doesn't make it clear to me why `operator delete[]` is relevant.

Since this is causing clang to become more restrictive, the best thing to do might be to revert to whatever we used to do in this situation, and then make a followup change which specifically explains why clang is becoming more restrictive.

https://github.com/llvm/llvm-project/pull/133950