[PATCH] D50949: Handle lambdas in .pcm [de]serialization
Steve O'Brien via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sun Aug 19 13:38:04 PDT 2018
elsteveogrande created this revision.
elsteveogrande added a reviewer: rsmith.
Herald added a subscriber: cfe-commits.
- Drop a couple of asserts preventing this (in the merge function)
- Merge all fields as usual
- Ensure select fields match, for ODR checking
- add previously-failing test to merge-lambdas.cpp
Test case by Andrew Gallagher
Test Plan: Andrew's repro from https://bugs.llvm.org/show_bug.cgi?id=38531 broke before this change, works after.
Repository:
rC Clang
https://reviews.llvm.org/D50949
Files:
include/clang/AST/LambdaCapture.h
lib/Serialization/ASTReaderDecl.cpp
test/Modules/merge-lambdas.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D50949.161406.patch
Type: text/x-patch
Size: 3781 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180819/5d1c5aa5/attachment.bin>
More information about the cfe-commits
mailing list