[PATCH] D110289: [libc++] Remove unused macro in __config

2021-09-23 Thread Louis Dionne via Phabricator via cfe-commits
This revision was not accepted when it landed; it landed in state "Needs Review". This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGe6126faba060: [libc++] Remove unused macro in __config (authored b

[PATCH] D110289: [libc++] Remove unused macro in __config

2021-09-23 Thread Louis Dionne via Phabricator via cfe-commits
ldionne accepted this revision as: libc++. ldionne added a comment. Other CI failures appear unrelated to the change, so I'm shipping this. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110289/new/ https://reviews.llvm.org/D110289

[PATCH] D110289: [libc++] Remove unused macro in __config

2021-09-23 Thread Louis Dionne via Phabricator via cfe-commits
ldionne added a comment. In D110289#3016525 , @jloser wrote: > I think it's used in a few places from a quick `git grep`: [...] `__MAC_OS_X_VERSION_MIN_REQUIRED` is defined by Apple's ``. It is equivalent to `__ENVIRONMENT_MAC_OS_X_VERSION_MIN_REQUIRED_

[PATCH] D110289: [libc++] Remove unused macro in __config

2021-09-23 Thread Louis Dionne via Phabricator via cfe-commits
ldionne updated this revision to Diff 374518. ldionne added a comment. Herald added subscribers: cfe-commits, llvm-commits, dexonsmith, usaxena95, kadircet, arphaman, hiraditya. Herald added projects: LLVM, clang-tools-extra. Remove uses of __MAC_OS_X_VERSION_MIN_REQUIRED in LLVM Repository: