smeenai added inline comments.

================
Comment at: clang-tools-extra/test/CMakeLists.txt:81
 
-add_lit_testsuite(check-clang-tools "Running the Clang extra tools' regression 
tests"
+add_lit_testsuite(check-clang-tools-extra "Running the Clang tools extra' 
regression tests"
   ${CMAKE_CURRENT_BINARY_DIR}
----------------
Nit: You need an explicit "'s" now (as in "Clang tools extra's regression 
tests"), since "extra" doesn't end with an s.


================
Comment at: clang-tools-extra/test/CMakeLists.txt:87
 
-set_target_properties(check-clang-tools PROPERTIES FOLDER "Clang extra tools' 
tests")
+set_target_properties(check-clang-tools-extra PROPERTIES FOLDER "Clang tools 
extra' tests")
 
----------------
Same here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97817

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to