https://github.com/Xazax-hun approved this pull request.

Looks good to me. I wonder if this also covers lambda captures (are they 
represented as fields). 

Also, I think it might be worth to add a comment that this functionality is 
covered by tests in a subsequent PR (or maybe we could even land them 
together). But this definitely made reviewing easier! 

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

Reply via email to