[PATCH] D126907: Deferred Concept Instantiation Implementation Take 2

[Erich Keane via Phabricator via cfe-commits]

erichkeane added a comment.

In D126907#3568943 <https://reviews.llvm.org/D126907#3568943>, @ChuanqiXu wrote:

> In D126907#3566633 <https://reviews.llvm.org/D126907#3566633>, @erichkeane wrote:
>
>> All of the comments from @ChuanqiXu done, thank you so much!
>>
>> As far as the crash you're seeing: That is typically a 'depth mismatch' kinda thing.  They sadly only show up when the types of the template arguments are different (that is, pack vs integral vs type), but I thought I got all of them with examples.
>>
>> I'll keep hunting for more during the day today, but any amount of reproducer you can provide (even if not minimized) would be incredibly helpful!
>
> Oh, if a not minimized reproducer is desired, you could find it at: https://github.com/alibaba/async_simple. It should crash when we run 'make' in build directory. (The version of libstdc++ is 10.2 in my environment)
>
> I would try to make a reduced example in the next week if we don't go on smoothly.

I was able to reproduce it, but my time today is pretty limited (I have an EWG meeting to chair today + prep + stuff after), so I'm not sure what progress I'll make today.  So if you make any progress on it, it would be greatly appreciated too!


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

https://reviews.llvm.org/D126907