[clang] [Clang,debuginfo] added vtt parameter in destructor DISubroutineType (PR #130674)

[via cfe-commits]

Markus =?utf-8?q?Gschoßmann?= <markus.gschossmann at tum.de>,
Markus =?utf-8?q?Gschoßmann?= <markus.gschossmann at tum.de>,
Markus =?utf-8?q?Gschoßmann?= <markus.gschossmann at tum.de>,
Markus =?utf-8?q?Gschoßmann?= <markus.gschossmann at tum.de>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/130674 at github.com>


================
@@ -0,0 +1,16 @@
+// RUN: %clang_cc1 -triple x86_64-none-linux-gnu -emit-llvm -debug-info-kind=limited %s -o - | FileCheck %s
----------------
mgschossmann wrote:

The key change of this pull request is that debug info for destructors is now taken from `FNType` instead of `D->getType()` / `Method->getType()`. `Method->getType()` does not include the implicit arguments, while `FNType` does. `FNType` however also includes the `this` argument. In order to prevent duplicate emission of `this`, the pull request introduces the `SkipFirst` argument in `getOrCreateInstanceMethodType`.

I now also added a test for MSVC, which checks for presence of the `should_call_delete` arg. It also enforces that the return value be `void*`. The new test seems to pass, however there seems to be an issue in the CI, so it is reported as failed. 

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