philnik777 wrote:

> > > > @ZequanWu I can't successfully build your reproducer with clang trunk. 
> > > > Would it be possible to provide the full test case or a fully reduced 
> > > > one?
> > > 
> > > 
> > > Because I reverted this change at 
> > > [567b2c6](https://github.com/llvm/llvm-project/commit/567b2c608c307c097315dd5ec4d6a5bbcddf898d),
> > >  so it no longer crashes. If you recommit this or checkout to 
> > > [5b36348](https://github.com/llvm/llvm-project/commit/5b363483cf2461617fbb2449491c9914811c8d53),
> > >  you can repro the crash.
> > 
> > 
> > No, I mean it isn't a well-formed program. There are semicolons missing 
> > after class definitions.
> 
> That's caused by creduce but it shouldn't matter (clang shouldn't crash due 
> to syntax errors). I attached the original crash source here as creduce is 
> still running. The command to repro is same as above. 
> [rtp_transmission_manager-8cce12.txt](https://github.com/user-attachments/files/16041931/rtp_transmission_manager-8cce12.txt)

Yes, but it's a lot easier to reduce a file without errors, at least for me. 
(And FWIW I'd also question reverting a bug fix that could result into bad code 
gen because of a crash-on-invalid it introduced)

https://github.com/llvm/llvm-project/pull/93113
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to