[PATCH] D109411: [clang] Enable the special enable_if_t diagnostics for libc++'s _EnableIf as well.

2021-09-08 Thread Louis Dionne via Phabricator via cfe-commits
ldionne added a comment. Actually, scratch that, we could tweak this patch so it accepts `__enable_if_t` as well (that will be the new spelling for `enable_if_t` for pre-C++14 places going forward. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D109411: [clang] Enable the special enable_if_t diagnostics for libc++'s _EnableIf as well.

2021-09-08 Thread Louis Dionne via Phabricator via cfe-commits
ldionne added a comment. This shouldn't be necessary anymore since we won't be using `_EnableIf` in libc++ going forward. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109411/new/ https://reviews.llvm.org/D109411

[PATCH] D109411: [clang] Enable the special enable_if_t diagnostics for libc++'s _EnableIf as well.

2021-09-08 Thread David Blaikie via Phabricator via cfe-commits
dblaikie added a comment. Would at. least need a test case, but otherwise sounds OK to make things a bit better. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109411/new/ https://reviews.llvm.org/D109411

[PATCH] D109411: [clang] Enable the special enable_if_t diagnostics for libc++'s _EnableIf as well.

2021-09-07 Thread Arthur O'Dwyer via Phabricator via cfe-commits
Quuxplusone created this revision. Quuxplusone added reviewers: ldionne, doug.gregor, rsmith, dblaikie, courbet, saar.raz. Quuxplusone added a project: clang. Quuxplusone requested review of this revision. Herald added a subscriber: cfe-commits. This comes from lengthy discussion between