[PATCH] D151320: [clang] Add `// expected-maybe-no-diagnostics` comment to VerifyDiagnosticConsumer

Vlad Serebrennikov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed May 24 05:22:29 PDT 2023


Endill created this revision.
Endill added reviewers: aaron.ballman, hfinkel, dblaikie.
Herald added a project: All.
Endill requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.

This patch introduces a `// expected-maybe-not-diagnostic` that silence warnings about missing `// expected-no-diagnostic`, but doesn't conflict with `// expected-error` directives.

I go into details in the Discourse thread <https://discourse.llvm.org/t/rfc-opt-in-way-to-make-verifydiagnosticconsumer-slightly-less-strict/70747> about why we need this, and why we need this in the form implemented.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D151320

Files:
  clang/include/clang/Frontend/VerifyDiagnosticConsumer.h
  clang/lib/Frontend/VerifyDiagnosticConsumer.cpp
  clang/test/Frontend/verify-maybe-no-diagnostics.c

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D151320.525115.patch
Type: text/x-patch
Size: 4860 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20230524/bcf31238/attachment.bin>


More information about the cfe-commits mailing list