[PATCH] D40995: [TextDiagnosticBuffer] Fix diagnostic note emission order.

2017-12-15 Thread Hal Finkel via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rC320904: [TextDiagnosticBuffer] Fix diagnostic note emission order (authored by hfinkel, committed by ). Repository: rC Clang https://reviews.llvm.org/D40995 Files: include/clang/Frontend/TextDiagnos

[PATCH] D40995: [TextDiagnosticBuffer] Fix diagnostic note emission order.

2017-12-08 Thread Joel E. Denny via Phabricator via cfe-commits
jdenny added a comment. Hi Richard. Thanks for accepting. I don't have commit privileges. Would you please commit for me? https://reviews.llvm.org/D40995 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/

[PATCH] D40995: [TextDiagnosticBuffer] Fix diagnostic note emission order.

2017-12-08 Thread Richard Smith - zygoloid via Phabricator via cfe-commits
rsmith accepted this revision. rsmith added a comment. This revision is now accepted and ready to land. Huh, I'm amazed we've had this bug for so long. Thanks! https://reviews.llvm.org/D40995 ___ cfe-commits mailing list cfe-commits@lists.llvm.org h

[PATCH] D40995: [TextDiagnosticBuffer] Fix diagnostic note emission order.

2017-12-07 Thread Joel E. Denny via Phabricator via cfe-commits
jdenny created this revision. The frontend currently groups diagnostics from the command line according to diagnostic level, but that places all notes last. Fix that by emitting such diagnostics in the order they were generated. https://reviews.llvm.org/D40995 Files: include/clang/Frontend/T