Szelethus added inline comments.
================ Comment at: clang/include/clang/Basic/AttrDocs.td:1189 +``onwership_returns``: Functions with this annotation return dynamic memory. +The second annotation parameter is the size of the returned memory in bytes. + ---------------- aaron.ballman wrote: > I'm a bit confused on this last bit; how is the user supposed to statically > know that value? I read this as claiming `my_malloc` returns 1 byte of > dynamic storage; can the user tie the allocation to a parameter value instead? I share your confusion, I guess. What I've written here is my best understanding of how this works, unfortunately, these annotations were made in ~2010 (when I was still in elementary school) and abandoned since. > I read this as claiming my_malloc returns 1 byte of dynamic storage; Thats what the corresponding code in the Static Analyzer leads me to believe as well. > can the user tie the allocation to a parameter value instead? No. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D156787/new/ https://reviews.llvm.org/D156787 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
