anthonyhatran wrote:

> Looks like we need to fix a few more things. Did you resolve the issue with 
> the existing test cases failing?

Apologies for the belated reply on this one; I ran the test suite overnight 
just to double-check that the test cases I listed were actually caused by my 
changes.

To answer your question,: yes, I have proposed solutions for most of them. 
Unfortunately I don't think there is just one adjustment that fixes all of them 
though (some include modifying the test cases themselves others involve 
changing what I added). I'll try to get the commit in as soon as possible so 
you can take a look.

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

Reply via email to