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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to