<div dir="ltr">Any feedback?</div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature">Thanks,<br>--Serge<br></div></div>
<br><div class="gmail_quote">2015-08-26 0:26 GMT+06:00 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">sepavloff created this revision.<br>
sepavloff added a subscriber: cfe-commits.<br>
<br>
Instantiation of static class members can be not obvious in some cases. Using<br>
modules can cause problems even more difficult to diagnose. PR24425 describes<br>
one of such cases. As a way to assist a user, compiler could issue warnings if<br>
code that is potentially erroneous. This patch implements a warning if a static<br>
class member is used in a module, but cannot be instantiated there.<br>
<br>
<a href="http://reviews.llvm.org/D12326" rel="noreferrer" target="_blank">http://reviews.llvm.org/D12326</a><br>
<br>
Files:<br>
  include/clang/Basic/DiagnosticGroups.td<br>
  include/clang/Basic/DiagnosticSemaKinds.td<br>
  lib/Sema/SemaDecl.cpp<br>
  test/Modules/Inputs/module-inst.cpp<br>
  test/Modules/Inputs/module-inst/a.h<br>
  test/Modules/Inputs/module-inst/module.modulemap<br>
<br>
</blockquote></div><br></div>