[PATCH] D82940: [ASTReader][ASTWriter][PCH][Modules] Fix (de)serialization of SwitchCases

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jul 1 04:17:21 PDT 2020


aaron.ballman requested changes to this revision.
aaron.ballman added a comment.
This revision now requires changes to proceed.

In D82940#2125005 <https://reviews.llvm.org/D82940#2125005>, @riccibruno wrote:

> In D82940#2124955 <https://reviews.llvm.org/D82940#2124955>, @aaron.ballman wrote:
>
> > LGTM, thank you for the fix!
>
>
> Wait, I am not sure that it is the right fix. The body of the lambda should not be different from the body of the `CXXMethodDecl`.


I'm not certain I understand, but I'm removing my LG while we figure it out. Can you expound a bit further?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D82940/new/

https://reviews.llvm.org/D82940





More information about the cfe-commits mailing list