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

Looks good to me, I think the commit is ready to be merged.

The only remaining question is whether we want to enable this feature by 
default -- I think it may be worth to wait a bit for other opinion(s?) about 
that question, but @ankurkraj I'm happy to help you with merging your commit 
whenever you request it.

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

Reply via email to