[libcxx-commits] [PATCH] D97904: [libcxx] updates the feature-test macro generator
Louis Dionne via Phabricator via libcxx-commits
libcxx-commits at lists.llvm.org
Fri Mar 5 06:14:45 PST 2021
ldionne accepted this revision.
ldionne added a comment.
This revision is now accepted and ready to land.
In D97904#2605361 <https://reviews.llvm.org/D97904#2605361>, @cjdb wrote:
> So are we addressing issues pre-merge or post-merge? A bit confused on this one, sorry.
My ask was to change to
"depends": "defined(__cpp_concepts) && __cpp_concepts >= 201907L",
since that's what we really mean. The other feedback about the `depends` + `internal_depends` naming is something we can (and should) address separately. Ship it once you've applied the changes explained just above.
Sorry about the lack of clarity in my review yesterday. I tried reviewing a lot of stuff before going OOO for an extended period of time and I think I went a little bit too fast. I should have explained better.
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the libcxx-commits