[llvm] [clang] [clang-tools-extra] [Clang] Fix : More Detailed "No expected directives found" (PR #78338)

Aaron Ballman via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 1 10:48:21 PST 2024


================
@@ -179,7 +179,7 @@ def err_verify_invalid_no_diags : Error<
     "%select{expected|'expected-no-diagnostics'}0 directive cannot follow "
     "%select{'expected-no-diagnostics' directive|other expected directives}0">;
----------------
AaronBallman wrote:

I think this is the same issue -- in both cases, it's a hard-coded `expected-no-diagnostics` that is incorrect in the presence of a different prefix being used. Because of that, I'd still recommend doing it as part of this PR instead of as a separate one.

https://github.com/llvm/llvm-project/pull/78338


More information about the cfe-commits mailing list