AdityaOP007 wrote: > Hm, I see, so `lifetime_capture_by` isn't supported for constructors, I don't > think we should remove that guard, it certainly isn't in the scope of this > PR, it would still be nice to keep the added test though. Also, I see that > you have some new binary file added, I think you should remove it too.
Thanks for the clarification! I've restored the original constructor guard, removed the now-redundant constructor-specific handling, kept the constructor regression test as suggested, and removed the accidentally added file. The rest of the implementation remains unchanged and focused on the original issue. Please let me know if there's anything else I should 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
