[PATCH] D154701: Overridden CXXMethodDecl::isVirtual() assertion failed before fully imported.

Ding Fei via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 7 04:04:07 PDT 2023


danix800 created this revision.
danix800 added a project: clang.
Herald added a project: All.
danix800 requested review of this revision.
Herald added a subscriber: cfe-commits.

For non-pure methods with no written `virtual`, `isVirtual()` tries to count size of overridden methods. For importing, size might not be correct before fully loaded.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D154701

Files:
  clang/lib/AST/DeclCXX.cpp
  clang/test/Analysis/ctu-astimport-virtual-assertion/Inputs/externalDefMap.txt
  clang/test/Analysis/ctu-astimport-virtual-assertion/Inputs/input.cpp
  clang/test/Analysis/ctu-astimport-virtual-assertion/Inputs/input.h
  clang/test/Analysis/ctu-astimport-virtual-assertion/main.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D154701.538067.patch
Type: text/x-patch
Size: 9147 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20230707/252ea3ae/attachment-0001.bin>


More information about the cfe-commits mailing list