<div>Thanks for the heads up Reid - i think i should be ok as long as VS2012 (takes even longer to link - ugh!) is supported.</div>
<div> </div>
<div>Faisal Vali<br><br></div><br><br>
<div class="gmail_quote">On Thu, Dec 26, 2013 at 12:02 PM, Reid Kleckner <span dir="ltr"><<a href="mailto:rnk@google.com" target="_blank">rnk@google.com</a>></span> wrote:<br>
<blockquote style="BORDER-LEFT:#ccc 1px solid;MARGIN:0px 0px 0px 0.8ex;PADDING-LEFT:1ex" class="gmail_quote">
<div dir="ltr">
<div class="gmail_extra">
<div class="gmail_quote">
<div class="im">On Tue, Dec 24, 2013 at 6:07 PM, Alp Toker <span dir="ltr"><<a href="mailto:alp@nuanti.com" target="_blank">alp@nuanti.com</a>></span> wrote:<br>
<blockquote style="BORDER-LEFT:#ccc 1px solid;MARGIN:0px 0px 0px 0.8ex;PADDING-LEFT:1ex" class="gmail_quote">
<div><br>On 25/12/2013 00:49, Faisal Vali wrote:<br>
<blockquote style="BORDER-LEFT:#ccc 1px solid;MARGIN:0px 0px 0px 0.8ex;PADDING-LEFT:1ex" class="gmail_quote">Hi Alp,<br>    This patch prevents clang from compiling in VS2010 (which supports strong enums) - that's because forward declared enums (which are being used in Diagnostic.h) aren't supported till VS2013.<br>
</blockquote><br></div>Looks like they're fine in VS2012, while VS2010 only had strongly /typed/ enums. Patched up in LLVM r197992.<br><br>Thanks for spotting this Faisal<br></blockquote>
<div><br></div></div>
<div>Faisal, we're going to move to C++11 in the next few months and drop 2010 support when that happens, so I recommend upgrading soon.</div></div></div></div></blockquote></div><br>