[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