https://github.com/NagyDonat commented:

Thanks for  implementing the changes that I suggested!

From my previous review the only question that remains is the open-ended one 
about the naming/role of the option `CheckedSwapFunctions`.

I also spotted one annoying little syntax error in the RST docs (see inline 
comment).

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

Reply via email to