[clang] [MS][clang] Add support for vector deleting destructors (PR #126240)

[Reid Kleckner via cfe-commits]

rnk wrote:

I'll try to take a look at this when I get a chance, but that looks like it's not happening today.

I read the comments a bit and found some codesearch links worth sharing for reference. Here's the original UnicodeString array new operation:
https://source.chromium.org/chromium/chromium/src/+/main:third_party/icu/source/common/filteredbrk.cpp;l=583;drc=e8c3bc9ea97d4423ad0515e5f1c064f486dae8b1

UnicodeString is a dynamic, polymorphic object with a virtual destructor, as one would expect. There are no DLLs in the build script.

I think removing the U_STATIC_IMPLEMENTATION define makes all these classes dllexport, which disables our vector/scalar alias optimization, hiding the problem.

More debugging is required.

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