shashi1687 wrote:

Hello @zwuis/everyone,

Thank you again for your thorough review. I’ve incorporated all suggestions:

* Moved the suppression guard into the `case AR_Deprecated:` block
* Used `dyn_cast_if_present` with a single `isImplicit() && isDefaulted()` check
* Combined constructor & destructor detection via `isa<…>`
* Cleaned up the release notes entry

All Sema tests (`check-clang`) pass locally. Could you please take another look 
when you have a chance? I really appreciate your time and guidance on this.

Thanks so much! 😊


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

Reply via email to