[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