[PATCH] D57032: [SemaCXX] Param diagnostic matches overload logic

S. B. Tam via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 22 05:59:56 PST 2019


cpplearner added inline comments.


================
Comment at: lib/Sema/SemaDecl.cpp:5092
     // The parameter types are identical
-    if (Context.hasSameType(DefParamTy, DeclParamTy))
+    if (Context.hasSameType(DefParamTy.getUnqualifiedType(),
+                            DeclParamTy.getUnqualifiedType()))
----------------
I see this is what `Sema::FunctionParamTypesAreEqual` does, but maybe both places should use `hasSameUnqualifiedType`?


Repository:
  rC Clang

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

https://reviews.llvm.org/D57032





More information about the cfe-commits mailing list