[Clang][CodeGen] Fix this argument type for certain destructors

[jacobly]

Summary

With the Microsoft ABI, some destructors need to offset a parameter to
get the derived this pointer, in which case the type of that parameter
should not be a pointer to the derived type.

Fixes #60465

[jacobly]

Added regression test specific to the issue.

[jacobly]

Cleanup and minor fixes.

[aaron.ballman]

Please be sure to add release notes to clang/docs/ReleaseNotes.rst

Things look reasonable to me, but it'd be nice if someone with more Windows ABI experience could take a look as well.

clang/lib/CodeGen/MicrosoftCXXABI.cpp

243

[jacobly]

Address review comments.

[efriedma]

If the parameter attributes are relevant, should we change the CHECK lines in the tests to check for them? At least for some of the tests. Once we fix these tests to enable opaque pointer types, you won't actually be checking for anything.

[jacobly]

Fix bad diff update.

[efriedma]

LGTM

[jacobly]

I was hoping @rnk was available for review, but otherwise this is ready to land and I don't have commit access.

[efriedma]

How do you want to be listed as author in the git commit (Name <email>)?

[jacobly]

Jacob Young <[e-mail]>

[efriedma]

rG674099113533

[efriedma]

Maybe worth cherry-picking to 16 branch? I think someone will need to rebase onto the branch for that, though; there was merge conflict on the microsoft-abi-eh-cleanups.cpp change.

[aaron.ballman]

In D143233#4160206, @efriedma wrote:

Maybe worth cherry-picking to 16 branch? I think someone will need to rebase onto the branch for that, though; there was merge conflict on the microsoft-abi-eh-cleanups.cpp change.

I wouldn't be opposed to picking this onto 16 given that it's fixing an ABI issue, if someone wants to do the work to cherry-pick it.

[jacobly]

In D143233#4161909, @aaron.ballman wrote:

In D143233#4160206, @efriedma wrote:

Maybe worth cherry-picking to 16 branch? I think someone will need to rebase onto the branch for that, though; there was merge conflict on the microsoft-abi-eh-cleanups.cpp change.

I wouldn't be opposed to picking this onto 16 given that it's fixing an ABI issue, if someone wants to do the work to cherry-pick it.

https://github.com/llvm/llvm-project/issues/61124 https://github.com/llvm/llvm-project-release-prs/pull/336