[PATCH] D25678: [modules] Do not report missing definitions of demoted constexpr variable templates.This is a followup to regression introduced in r284284.This should fix our libstdc++ modules builds.

[Manman Ren via cfe-commits]

manmanren added inline comments.


================
Comment at: lib/Sema/SemaDecl.cpp:10129
+        !Var->isThisDeclarationADemotedDefinition()) {
+      assert(Var->isThisDeclarationADemotedDefinition() && 
getLangOpts().Modules
+             && "Demoting decls is only in the contest of modules!");
----------------
Is the logic correct here? The if statement says 
!Var->isThisDeclarationADemotedDefinition(), and we then assert 
Var->isThisDeclarationADemotedDefinition() && getLangOpts().Modules.


Repository:
  rL LLVM

https://reviews.llvm.org/D25678



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits