AdityaOP007 wrote:

> Looks much better now, but you should probably delete the added binary file.

Thanks for the review! I cleaned up the patch by removing the accidentally 
added binary file and reducing the changes to only what's necessary for this 
issue. I also restored the !isa<CXXConstructorDecl>(FD) check as suggested and 
added a focused regression test covering constructor handling. I've updated the 
PR description with the AI tool use disclosure in accordance with LLVM's 
policy. Please let me know if there's anything else you'd like me to adjust.


https://github.com/llvm/llvm-project/pull/212264
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to