[PATCH] D152259: [Clang] Make increment bool SFINAE-friendly

[Erich Keane via Phabricator via cfe-commits]

erichkeane added a comment.

This should have a release note.

I think the bug is asking to make this a SFINAE failure (it isn't clear what you mean by "SFINAE Friendly").  Our mechanism for that is to mark the diagnostic SFINAEError.



================
Comment at: clang/lib/Sema/SemaExpr.cpp:14603
+      // Ensure increment bool SFINAE-friendly.
+      if (S.isUnevaluatedContext() || S.isSFINAEContext())
+        return QualType();
----------------
I think what you really need to do is just set `ext_increment_bool` to be `SFINAEError` in `DiagnosticSemaKinds.td`, right?  Not all of this?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D152259