[PATCH] D76696: [AST] Build recovery expressions by default for C++.

Hubert Tong via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 26 12:29:32 PDT 2020


hubert.reinterpretcast added a comment.

> In D76696#1944014 <https://reviews.llvm.org/D76696#1944014>, @hubert.reinterpretcast wrote:
> 
>> We have also encountered crashes in downstream testing caused by this using just the vanilla source from trunk. When there is a proposed fix, please let us know so we can test. Thanks.
> 
> 
> We can try but it would be very useful to have details, no idea whether it's going to be the same issue and the more we know about the easier it is to improve the design.
>  Can you describe the case at all? Any chance of a minimal example? Ideally we'd check it in.

All of them appear to have unresolved names in constexpr evaluation. It is likely to be the same issue.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76696





More information about the cfe-commits mailing list