[PATCH] Add clang-format-fuzzer target

Alexey Samsonov vonosmas at gmail.com
Tue Jan 27 15:22:56 PST 2015


Looks good, but I'll let clang-format owners approve this.


================
Comment at: tools/clang-format/CMakeLists.txt:8
@@ -7,3 +7,3 @@
 target_link_libraries(clang-format
   clangBasic
   clangFormat
----------------
Maybe, we could introduce
  set(CLANG_FORMAT_LIB_DEPS
    clangBasic
    clangFormat
    clangRewrite
    clangToolingCore
    )
and use ${CLANG_FORMAT_LIB_DEPS} in both clang-format and clang-format-fuzzer. I have no strong preference, though.

================
Comment at: tools/clang-format/fuzzer/ClangFormatFuzzer.cpp:1
@@ +1,2 @@
+//===-- clang-format/ClangFormatFuzzer.cpp - Fuzz the Clang format tool ---===//
+//
----------------
clang-format/fizzer/ClangFormatFuzzer.cpp ?

http://reviews.llvm.org/D7202

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list