Xinlong-Chen wrote: > Thank you for the fix! Please also add a release note. > > Note, your summary is what goes into the commit message usually. I don't > think the output is necessary in the summary but the summary should include a > description of the fix. > > If you have more to say that does not make sense to go into the commit > message they can always be added as a comment to the PR for more details for > reviewers.
Thanks for the feedback! I've updated the PR summary to better describe the root cause and the fix, and added a release note as well. https://github.com/llvm/llvm-project/pull/183707 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
