https://github.com/haoNoQ approved this pull request.
Ooo excellent LGTM! We already do this for const pointers, you've just extended it to known standard "inout" parameters. This could probably be, like, a very useful compiler attribute. "This buffer is NOT used purely for output purposes. It needs to be initialized with some actual data before invoking the function." Then we wouldn't need to maintain a list, and folks would be able to annotate their custom functions this way. It could also be consumed by compiler warnings such as `-Wuninitialized` where hardcoding function names is somewhat frowned upon even when the functions are standard. https://github.com/llvm/llvm-project/pull/173854 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
