[PATCH] D58920: [Modules][PR39287] Consolidate multiple std's
Brian Gesiak via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue May 21 08:19:25 PDT 2019
modocache updated this revision to Diff 200518.
modocache added a comment.
Hmm... alright, I'm not really sure how I could implement a test that fails without this, but I added a check in the FindExistingResult destructor.
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D58920/new/
https://reviews.llvm.org/D58920
Files:
lib/Serialization/ASTReaderDecl.cpp
test/Modules/Inputs/mod-virtual-destructor-bug-two/a.h
test/Modules/Inputs/mod-virtual-destructor-bug-two/module.modulemap
test/Modules/Inputs/mod-virtual-destructor-bug/a.h
test/Modules/Inputs/mod-virtual-destructor-bug/module.modulemap
test/Modules/mod-virtual-destructor-bug-two.cpp
test/Modules/mod-virtual-destructor-bug.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D58920.200518.patch
Type: text/x-patch
Size: 3612 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20190521/302e4ce3/attachment.bin>
More information about the cfe-commits
mailing list