steakhal wrote:

I took the time to update the branch with my recommendations.
Please confirm if you agree with the current status.

tldr; I only made sure the tests actually test what they intended now that the 
default threshold has changed.
I also added a couple more tests where it made sense, along with a new test 
showcasing the original issue that motivated the presence of the now disabled 
thresholds.

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

Reply via email to