https://github.com/Sirraide approved this pull request.

Alright, this does look like we’re handling all cases now (that I can think of 
at least).

However, I’d like to see some comments added to `f1`–`f3`, indicating why 
they’re dependent, because it took me a bit of thinking to figure that out just 
now...

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

Reply via email to