[libcxx-commits] [PATCH] D151900: [libc++][NFC] Create a new folder for config-related headers.

Konstantin Varlamov via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Wed Jun 7 02:52:55 PDT 2023


var-const added a comment.

In D151900#4387750 <https://reviews.llvm.org/D151900#4387750>, @Mordante wrote:

> Could you add some information in the commit message why this is a good idea?

Added some comments to the patch description, please let me know what you think. This came out of my first attempts to add macros for configuring hardening modes. I find `__config` hard to navigate due to its size, and it seems to have a few "themes" that seem like a good fit to be split into separate, more focused headers. I thought to refactor `__config` first before adding even more stuff to it. I also hope that having more focused headers (e.g. `__config_dir/language_support.h`, `__config_dir/abi.h`) would make finding macros easier.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D151900



More information about the libcxx-commits mailing list