AaronBallman wrote:

> @AaronBallman Thanks for the review. Sorry for responding so late - I've 
> missed your review.
> 
> The tests aren't actually unrelated to the path. This is the file where I 
> discovered the crash, so I just used it to demonstrate that it's fixed. I'm 
> not sure how to split things, since I can't add the run line without fixing 
> the bug.

Ah, I see now what's going on, and yeah, the tests are not unrelated to the 
patch (the added RUN line is what causes the crash today), sorry for missing 
that.

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

Reply via email to