mbonadei accepted this revision. mbonadei added a comment. This revision is now accepted and ready to land.
LGTM, just a couple of comments. ================ Comment at: llvm/utils/gn/secondary/clang-tools-extra/clangd/xpc/BUILD.gn:17 + configs += [ "//llvm/utils/gn/build:clang_code" ] + deps = [ + "//clang-tools-extra/clangd", ---------------- Missing ":conversions" in deps. ================ Comment at: llvm/utils/gn/secondary/clang-tools-extra/clangd/xpc/enable.gni:3 + # Whether to build clangd's XPC components. + clangd_build_xpc = current_os == "mac" +} ---------------- Is it ok to keep this ` = current_os == "mac"`, from the CL description I had the feeling that xpc tests were still disabled. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60124/new/ https://reviews.llvm.org/D60124 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
