From d76650bced1de20cb79d1b4a0c1732339a5d0099 Mon Sep 17 00:00:00 2001
From: Owen Pan <owenpiano@gmail.com>
Date: Sat, 7 Dec 2024 00:16:42 -0800
Subject: [PATCH] [clang-format] Clean up cmake target
 clang-format-check-format

---
 clang/lib/Format/CMakeLists.txt | 42 +++++++++++++++++----------------
 1 file changed, 22 insertions(+), 20 deletions(-)

diff --git a/clang/lib/Format/CMakeLists.txt b/clang/lib/Format/CMakeLists.txt
index 7bb1a44f546f138..7d2db087328e4ad 100644
--- a/clang/lib/Format/CMakeLists.txt
+++ b/clang/lib/Format/CMakeLists.txt
@@ -41,31 +41,33 @@ file(GLOB_RECURSE files
 
 set(check_format_depends)
 set(i 0)
-foreach (file IN LISTS files)
-  add_custom_command(OUTPUT clang-format-check-format${i}
+foreach(file IN LISTS files)
+  add_custom_command(OUTPUT check_format_depend_${i}
     COMMAND clang-format ${file} | diff -u ${file} -
     VERBATIM
-    COMMENT "Checking format of ${file}..."
+    COMMENT "Checking format of ${file}"
     )
-  list(APPEND check_format_depends clang-format-check-format${i})
+  list(APPEND check_format_depends check_format_depend_${i})
 
   math(EXPR i ${i}+1)
-endforeach ()
+endforeach()
 
 add_custom_target(clang-format-check-format DEPENDS ${check_format_depends})
 
-if(CLANG_INCLUDE_DOCS)
-  set(style_options_depends ${CMAKE_CURRENT_BINARY_DIR}/dummy_output)
-  set(docs_tools_dir ${CLANG_SOURCE_DIR}/docs/tools)
-  add_custom_command(OUTPUT ${style_options_depends}
-    COMMAND ${Python3_EXECUTABLE} dump_format_style.py &&
-            touch ${style_options_depends}
-    WORKING_DIRECTORY ${docs_tools_dir}
-    DEPENDS ${CLANG_SOURCE_DIR}/include/clang/Format/Format.h
-            ${CLANG_SOURCE_DIR}/include/clang/Tooling/Inclusions/IncludeStyle.h
-            ${CLANG_SOURCE_DIR}/docs/ClangFormatStyleOptions.rst
-            ${docs_tools_dir}/plurals.txt
-            ${docs_tools_dir}/dump_format_style.py
-    )
-  add_custom_target(clang-format-style-options DEPENDS ${style_options_depends})
-endif()
+set(style_options_depends ${CMAKE_CURRENT_BINARY_DIR}/dummy_output)
+set(docs_tools_dir ${CLANG_SOURCE_DIR}/docs/tools)
+set(style_options_rst ${CLANG_SOURCE_DIR}/docs/ClangFormatStyleOptions.rst)
+add_custom_command(OUTPUT ${style_options_depends}
+  COMMAND ${Python3_EXECUTABLE} dump_format_style.py &&
+          touch ${style_options_depends}
+  WORKING_DIRECTORY ${docs_tools_dir}
+  VERBATIM
+  COMMENT "Updating ${style_options_rst}"
+  DEPENDS ${CLANG_SOURCE_DIR}/include/clang/Format/Format.h
+          ${CLANG_SOURCE_DIR}/include/clang/Tooling/Inclusions/IncludeStyle.h
+          ${style_options_rst}
+          ${docs_tools_dir}/plurals.txt
+          ${docs_tools_dir}/dump_format_style.py
+  )
+
+add_custom_target(clang-format-style-options DEPENDS ${style_options_depends})