[PATCH] D68554: [clang-format] Proposal for clang-format to give compiler style warnings

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 17 21:04:20 PDT 2019


MyDeveloperDay marked an inline comment as done.
MyDeveloperDay added inline comments.


================
Comment at: clang/test/Format/dry-run-alias.cpp:1-2
+// RUN: clang-format -style=LLVM -i -n %s 2> %t.stderr
+// RUN: grep -E "*code should be clang-formatted*" %t.stderr
+
----------------
hliao wrote:
> Why not check with `FileCheck`?
Absolutely no real reason other than my lack of ability to get it working correctly. I was trying to follow the other examples of how it could be written with FIleCheck but failed.  In the end, I copied how it was done in some of the other tests for now but If someone could help me I would love to change them to use FileCheck properly.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D68554





More information about the cfe-commits mailing list