tahonermann added a comment. > To me, an important metric of a quality patch is if the title and the summary > describe why we have this patch, and what we do about it.
@steakhal, I'll work with the people submitting these patches to add more context to the title, summary, and commit comments. I agree that would be helpful. Ideally, we would copy the report from the static analysis tool, but we have been informed that we are not allowed to divulge the tool we are using, so we aren't permitted to do that. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D158285/new/ https://reviews.llvm.org/D158285 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits