<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Apr 11, 2017 at 6:59 AM, Alex Lorenz via Phabricator via cfe-commits <span dir="ltr"><<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">arphaman added a comment.<br>
<br>
Ah, I see. I guess if `static` is removed the warning will go away. Thanks!<div class="gmail-HOEnZb"><div class="gmail-h5">
<br>
<a href="https://reviews.llvm.org/D29877" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D29877</a></div><div class="gmail-h5"><br></div></div></blockquote><div><br></div><div>I would have appreciated a heads-up here, since libc++ no longer builds with warnings-as-errors.</div><div><br></div><div>include/memory:</div><div><br></div><div><div>namespace __has_pointer_type_imp</div><div>{</div><div> template <class _Up> static __two __test(...);</div><div> template <class _Up> static char __test(typename _Up::pointer* = 0);</div><div>}</div><div><br></div><div>template <class _Tp></div><div>struct __has_pointer_type</div><div> : public integral_constant<bool, sizeof(__has_pointer_type_imp::__test<_Tp>(0)) == 1></div><div>{</div><div>};</div></div><div><br></div><div>That first function is not really "unused" (though clang now claims that it is)</div><div><br></div><div>-- Marshall</div><div> </div></div><br></div></div>