[libc-commits] [PATCH] D77340: [libc] Add fully-qualified target names.
Siva Chandra via Phabricator via libc-commits
libc-commits at lists.llvm.org
Thu Apr 2 15:44:15 PDT 2020
sivachandra added a comment.
This change implements the topic I brought up many weeks ago: http://lists.llvm.org/pipermail/libc-dev/2020-January/000038.html.
It is probably motivated by my personal preference largely driven by my experience working with GN and Bazel. I am open to listening to opinions from others as it does add verbosity to the target names and can be viewed as a development overhead. But, it also eliminates ambiguity in naming targets, and makes it clear and explicit what each target is about when one reads the target names. More importantly, I like the fact the we can now set up directory specific build and run targets as I explained in the above post (for example, `test.src.signal.__build__` and `test.src.signal.__run__`.)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D77340/new/
https://reviews.llvm.org/D77340
More information about the libc-commits
mailing list