phosek added inline comments.
================ Comment at: llvm/utils/gn/secondary/llvm/utils/unittest/unittest.gni:34 + # out/gn/obj/clang/unittests/Format/FormatTests, which seems fine. + output_dir = target_out_dir + deps += [ ---------------- What if someone explicitly sets `output_dir` in the invoker? We should either preserve that value or assert with an error message that overriding `output_dir` in `unittest`s is unsupported. ================ Comment at: llvm/utils/gn/secondary/llvm/utils/unittest/unittest.gni:35 + output_dir = target_out_dir + deps += [ + "//llvm/utils/unittest/UnitTestMain", ---------------- You should check if `deps` is defined and set `deps = []` if not, otherwise the attempt to add to non-existent list is going to fail. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D56116/new/ https://reviews.llvm.org/D56116 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
