[PATCH] D144285: [Clang] Implement CWG2518 - static_assert(false)

Erich Keane via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 17 11:41:23 PST 2023


erichkeane accepted this revision.
erichkeane added inline comments.
This revision is now accepted and ready to land.


================
Comment at: clang/lib/Sema/SemaDeclCXX.cpp:16806
+    bool InTemplateDefinition =
+        getLangOpts().CPlusPlus && CurContext->isDependentContext();
+
----------------
CplusPlus check is now not really beneficial?  I'm not sure how much it matters now though that these are both just bit-loads.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144285



More information about the cfe-commits mailing list