[PATCH] D82657: [AST][RecoveryAST] Preserve the type by default for recovery expression.

guopeilin via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Aug 22 18:20:34 PDT 2021


guopeilin added a comment.

In D82657#2956851 <https://reviews.llvm.org/D82657#2956851>, @sammccall wrote:

> In D82657#2956751 <https://reviews.llvm.org/D82657#2956751>, @guopeilin wrote:
>
>> Hi @hokein , I encounter a bug when clang parses enum and I have been recorded in https://bugs.llvm.org/show_bug.cgi?id=51554.
>
> I added a comment on the bug and sent https://reviews.llvm.org/D108451
>
> This is a "rough edge" between erroneous and well-formed code and I'm not totally sure it's the perfect fix, but it does seem to be robust against crashes which is definitely he biggest thing.

Yes, that's great, thanks a lot.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82657



More information about the cfe-commits mailing list