<div dir="ltr">Sorry about the trouble.  Fixed in r282761. </div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Sep 29, 2016 at 1:16 PM, Adrian McCarthy <span dir="ltr"><<a href="mailto:amccarth@google.com" target="_blank">amccarth@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Sure, I'll post a fix in a few minutes.  Running tests now...</div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Sep 29, 2016 at 12:58 PM, Adrian Prantl <span dir="ltr"><<a href="mailto:aprantl@apple.com" target="_blank">aprantl@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span><br>
> On Sep 20, 2016, at 10:20 AM, Adrian McCarthy via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br>
> +<br>
> +  // Some Microsoft tools, like Binscope, expect a backend version number of at<br>
> +  // least 8.something, so we'll coerce the LLVM version into a form that<br>
> +  // guarantees it'll be big enough without really lying about the version.<br>
> +  Version BackVer = {<br>
> +      1000 * LLVM_VERSION_MAJOR +<br>
> +      10 * LLVM_VERSION_MINOR +<br>
> +      LLVM_VERSION_PATCH,<br>
> +      0, 0, 0 };<br>
> +  OS.AddComment("Backend version");<br>
> +  for (int N = 0; N < 4; ++N)<br>
> +    OS.EmitIntValue(BackVer.Part[N<wbr>], 2);<br>
> +<br>
<br>
</span>This happens to break some of our internal buildbots. Some of them are configured to set LLVM_VERSION_MAJOR to 9999 which doesn't fit into the allocated 2 bytes when multiplied with 1000 and asserts in OS.EmitIntValue. Since we don't impose any limits on LLVM_VERSION_MAJOR it would be good to handle this gracefully here. Could you please take a look? I'm fine with clamping the number to 0xffff or something like that.<br>
<br>
thanks!<br>
<span class="m_6179848541875785456HOEnZb"><font color="#888888">-- adrian<br>
</font></span></blockquote></div><br></div>
</div></div></blockquote></div><br></div>