================
@@ -2472,13 +2473,23 @@ bool 
Sema::CheckTemplateParameterList(TemplateParameterList *NewParams,
         NewDefaultLoc = NewTypeParm->getDefaultArgumentLoc();
         SawDefaultArgument = true;
 
-        if (!OldTypeParm->getOwningModule())
+        bool SameDefault =
+            getASTContext().isSameDefaultTemplateArgument(OldTypeParm,
+                                                          NewTypeParm);
+        if (Module *ImportedM = OldTypeParm->getImportedOwningModule()) {
+          if (!SameDefault) {
+            InconsistentDefaultArg = true;
+            PrevModuleName = ImportedM->getFullModuleName();
+          }
+        } else if (Module *LocalM = OldTypeParm->getLocalOwningModule()) {
----------------
ChuanqiXu9 wrote:

```suggestion
        } else if (Module *LocalM = OldTypeParm->getLocalOwningModule(); LocalM 
&& LocalM->isModuleMapModule()) {
        RedundantDefaultArg = !SameDefault;
}
```

nit: 

https://github.com/llvm/llvm-project/pull/185237
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to