<div dir="ltr">MSVC accepts<div><div><br></div><div>static int x7;</div><div>extern __declspec(selectany) int x7;</div></div><div><br></div><div style>But it also accepts </div><div><br></div><div>static int x7;</div><div>
extern int x7;</div><div><br></div><div style>That seems like a separate issue.  It doesn't seem that important to me to accept this.</div><div style><br></div><div style>Clang errors out with 'attribute declaration must precede definition', which doesn't seem useful to have in that test.</div>
<div style><br></div><div style>OK to commit as is?</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, May 17, 2013 at 3:36 PM, Rafael Ávila de Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
  Should<br>
<br>
  static int x;<br>
  extern __declspec(selectany) int x;<br>
<br>
  work or be rejected? If rejected, LGTM with it added as a test.<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D814" target="_blank">http://llvm-reviews.chandlerc.com/D814</a><br>
<br>
BRANCH<br>
  selectany<br>
<br>
ARCANIST PROJECT<br>
  clang<br>
</blockquote></div><br></div>