[MS] Don't emit coverage for deleting dtors
authorReid Kleckner <rnk@google.com>
Tue, 26 Feb 2019 20:42:52 +0000 (20:42 +0000)
committerReid Kleckner <rnk@google.com>
Tue, 26 Feb 2019 20:42:52 +0000 (20:42 +0000)
commitf9ef9f868c82646203581d8808300874066fa14a
tree154d4a05662255119936b529d3b3763cb86aac17
parent721eaeff3a2429fecbaeb9a47ca1339cd7cb03ce
[MS] Don't emit coverage for deleting dtors

Summary:
The MS C++ ABI has no constructor variants, but it has destructor
variants, so we should move the deleting destructor variant check
outside the check for "does the ABI have constructor variants".

Fixes PR37561, so basic code coverage works on Windows with C++.

Reviewers: vsk

Subscribers: jdoerfert, cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D58691

llvm-svn: 354924
clang/lib/CodeGen/CodeGenPGO.cpp
clang/test/Profile/cxx-abc-deleting-dtor.cpp [new file with mode: 0644]