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

I scanned over the patch and everything looks sane. I also like how this is 
more expressive w.r.t. what's allowed to change and the simpler function calls.

The only thing to beware of is that `const` member variables will transform 
moves into copies, but this is not a problem in this POD type of class.

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

Reply via email to