[PATCH] D58691: [MS] Don't emit coverage for deleting dtors

[Reid Kleckner via Phabricator via cfe-commits]

rnk created this revision.
rnk added a reviewer: vsk.
Herald added a subscriber: jdoerfert.
Herald added a project: clang.

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++.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D58691

Files:
  clang/lib/CodeGen/CodeGenPGO.cpp
  clang/test/Profile/cxx-abc-deleting-dtor.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D58691.188427.patch
Type: text/x-patch
Size: 4557 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20190226/ba815f1a/attachment.bin>