[PATCH] D79691: [clangd] Have suppression comments take precedence over warning-as-error

Kadir Cetinkaya via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon May 11 14:01:23 PDT 2020


kadircet accepted this revision.
kadircet added a comment.
This revision is now accepted and ready to land.

LGTM, thanks!

Could you also put `Fixes https://github.com/clangd/clangd/issues/375` in the commit message?



================
Comment at: clang-tools-extra/clangd/unittests/DiagnosticsTests.cpp:381
       int i = 3;
       double f = [[8]] / i;  // NOLINT // error-ok
     }
----------------
you can also drop the `// error-ok` now as there are no diags left.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D79691/new/

https://reviews.llvm.org/D79691





More information about the cfe-commits mailing list