erichkeane wrote:

> It looks like we have a number of approvals and I haven't seen any objections 
> yet.
> 
> Should we go ahead with merging this in the current state? If so, I'll need 
> help there since I don't have commit access yet. @tarunprabhu would you mind 
> volunteering to mash the merge button again?

I had 1 minor objection, see my concern on the `uninitialized` attribute 
removal.

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

Reply via email to