[PATCH] D30131: [profiling] PR31992: Don't skip interesting non-base constructors

Vedant Kumar via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Feb 20 18:30:34 PST 2017


vsk updated this revision to Diff 89151.
vsk retitled this revision from "[profiling] Don't skip non-base constructors if there is a virtual base (fixes PR31992)" to "[profiling] PR31992: Don't skip interesting non-base constructors".
vsk edited the summary of this revision.
vsk added a comment.

- Use IsConstructorDelegationValid per Alex's suggestion. This exposed the variadic constructor case which we had also missed previously.


https://reviews.llvm.org/D30131

Files:
  lib/CodeGen/CGClass.cpp
  lib/CodeGen/CodeGenFunction.h
  lib/CodeGen/CodeGenPGO.cpp
  test/Profile/Inputs/cxx-class.proftext
  test/Profile/cxx-class.cpp
  test/Profile/cxx-structors.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D30131.89151.patch
Type: text/x-patch
Size: 6115 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170221/632fa513/attachment.bin>


More information about the cfe-commits mailing list