This patch needs testcases.<div><br></div><div><div>Also, the way in which you're updating the type of the parameter doesn't look correct -- there's no reason to think that the qualifiers will be local.</div></div>
<div><br></div><div>We discussed cases like this at the WG21 meeting in Issaquah last week, and decided that cv-stripping should *not* be applied to dependent parameter types when determining whether two function templates are redeclarations. Therefore:</div>
<div><br></div><div>  template<typename T> void f(T);</div><div>  template<typename T> void f(const T);</div><div><br></div><div>... are not redeclarations (because, for instance, they have different parameter types when T = int[]).</div>
<div><br></div><div>That makes this problem easier to fix: if the parameter type within PatternDecl is not dependent, set the type of the parameter in the instantiated function to that type. Otherwise, leave it alone, since we already know it will match.</div>
<br><div>On Tue Feb 18 2014 at 6:02:55 AM, suyog sarda <<a href="mailto:sardask01@gmail.com">sardask01@gmail.com</a>> wrote:</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr">Gentle Ping !! Please help in reviewing the patch for bug 18275.</div><div dir="ltr"><br><div><br clear="all"><br>-- <br>With regards,<br>Suyog Sarda<br>
</div></div></blockquote>