[PATCH] D148601: [Clang] Handle Error message to output proper Prefix

2023-04-28 Thread Usman Akinyemi via Phabricator via cfe-commits
Unique_Usman added a comment. In D148601#4301977 , @aaron.ballman wrote: > Thank you for working on this! These changes should come with test coverage > (we have a handful of verifier tests in `clang/test/Frontend` -- you can add > a new test file

[PATCH] D148601: [Clang] Handle Error message to output proper Prefix

2023-04-27 Thread Usman Akinyemi via Phabricator via cfe-commits
Unique_Usman added a comment. In D148601#4301977 , @aaron.ballman wrote: > Thank you for working on this! These changes should come with test coverage > (we have a handful of verifier tests in `clang/test/Frontend` -- you can add > a new test file

[PATCH] D148601: [Clang] Handle Error message to output proper Prefix

2023-04-27 Thread Usman Akinyemi via Phabricator via cfe-commits
Unique_Usman added a comment. Thanks @tbaeder, I really appreciate your time. Name is Unique_Usman and my email is usmanakinyemi...@gmail.com As you mentioned, if anyone later chime in, I will be glad to update the issue back. Thank you. Repository: rG LLVM Github Monorepo CHANGES SINCE

[PATCH] D148601: [Clang] Handle Error message to output proper Prefix

2023-04-25 Thread Usman Akinyemi via Phabricator via cfe-commits
Unique_Usman added a comment. In D148601#4295719 , @tbaeder wrote: > In D148601#4279604 , @Unique_Usman > wrote: > >> In D148601#4279334 , @tbaeder >> wrote: >> >>> I

[PATCH] D148601: [Clang] Handle Error message to output proper Prefix

2023-04-25 Thread Usman Akinyemi via Phabricator via cfe-commits
Unique_Usman updated this revision to Diff 516762. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D148601/new/ https://reviews.llvm.org/D148601 Files: clang/include/clang/Basic/DiagnosticFrontendKinds.td clang/lib/Frontend/VerifyDiagnosticConsumer.cpp Index:

[PATCH] D148601: [Clang] Handle Error message to output proper Prefix

2023-04-19 Thread Usman Akinyemi via Phabricator via cfe-commits
Unique_Usman updated this revision to Diff 515050. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D148601/new/ https://reviews.llvm.org/D148601 Files: clang/include/clang/Basic/DiagnosticFrontendKinds.td clang/lib/Frontend/VerifyDiagnosticConsumer.cpp Index:

[PATCH] D148601: [Clang] Handle Error message to output proper Prefix

2023-04-19 Thread Usman Akinyemi via Phabricator via cfe-commits
Unique_Usman added a comment. In D148601#4279334 , @tbaeder wrote: > I am not 100% sure about the semantics of passing multiple prefixes, i.e. if > the error is emitted for all prefixes individually or if it's only emitted if > no `expected` line for