[prev in list] [next in list] [prev in thread] [next in thread] 

List:       cfe-commits
Subject:    [PATCH] D56116: [gn build] Make `ninja check-clang` also run Clang's unit tests
From:       Petr Hosek via Phabricator via cfe-commits <cfe-commits () lists ! llvm ! org>
Date:       2018-12-30 22:37:09
Message-ID: d65833a2e1b93a79d6021acf1a6de378 () localhost ! localdomain
[Download RAW message or body]

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


[prev in list] [next in list] [prev in thread] [next in thread]