<div dir="ltr">If there are no more objections, comments or recommendations, I will commit this patch.<div><br></div><div>Thanks,</div><div>--Serge</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/11/5 Serge Pavlov <span dir="ltr"><<a href="mailto:sepavloff@gmail.com" target="_blank">sepavloff@gmail.com</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Friendly ping.</div><div class="gmail_extra"><div><div class="h5"><br><br><div class="gmail_quote">2013/10/28 Serge Pavlov <span dir="ltr"><<a href="mailto:sepavloff@gmail.com" target="_blank">sepavloff@gmail.com</a>></span><br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>  Warn on duplicate function specifier<br>
<br>
</div>  It looks like things are more complicated here.<br>
  Thank you for the comment and for the nice example.<br>
  In the updated patch __forceinline is considered as a separate specifier,<br>
  but its presence is reported same as for inline.<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D2025" target="_blank">http://llvm-reviews.chandlerc.com/D2025</a><br>
<br>
CHANGE SINCE LAST DIFF<br>
  <a href="http://llvm-reviews.chandlerc.com/D2025?vs=5157&id=5195#toc" target="_blank">http://llvm-reviews.chandlerc.com/D2025?vs=5157&id=5195#toc</a><br>
<br>
Files:<br>
  include/clang/Sema/DeclSpec.h<br>
  lib/Parse/ParseDecl.cpp<br>
  lib/Sema/DeclSpec.cpp<br>
  test/Parser/MicrosoftExtensions.c<br>
  test/Sema/declspec.c<br>
  test/SemaCXX/explicit.cpp<br>
  test/SemaCXX/virtuals.cpp<br>
</blockquote></div><br><br clear="all"><div><br></div></div></div><span class="HOEnZb"><font color="#888888">-- <br>Thanks,<br>--Serge<br>
</font></span></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br>Thanks,<br>--Serge<br>
</div>