[PATCH] D56193: [gn build] Add some llvm/tools: lli, lli-child-target

[Petr Hosek via Phabricator via llvm-commits]

phosek added inline comments.


================
Comment at: llvm/utils/gn/secondary/BUILD.gn:11
+    "//llvm/tools/lli",
+    "//llvm/tools/lli/ChildTarget:lli-child-target",
     "//llvm/tools/llvm-cfi-verify",
----------------
You seem to be overriding the name of the target to match the name of the directory and then setting the `output_name` explicitly everywhere. Is there a reason why not to use the same approach here as well?


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

https://reviews.llvm.org/D56193