[PATCH] D61001: [clang-format][tests] Explicitly specify style in some tests

Rafael Stahl via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jul 10 01:15:19 PDT 2019


r.stahl marked 2 inline comments as done.
r.stahl added inline comments.


================
Comment at: test/Format/language-detection.cpp:2
 // RUN: grep -Ev "// *[A-Z0-9_]+:" %s \
-// RUN:   | clang-format -style=llvm -assume-filename=foo.js \
 // RUN:   | FileCheck -strict-whitespace -check-prefix=CHECK1 %s
----------------
lebedev.ri wrote:
> What's wrong with `-style=llvm`?
I looked at all the tests in that folder and saw that LLVM is the more common usage and unified it.


Repository:
  rC Clang

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

https://reviews.llvm.org/D61001





More information about the cfe-commits mailing list