<div dir="ltr">Resending since I sent to wrong list again.<br><br><div class="gmail_quote"><div dir="ltr">On Tue, Sep 22, 2015 at 10:12 AM Zachary Turner <<a href="mailto:zturner@google.com">zturner@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">In regards to this CL: <a href="http://llvm.org/viewvc/llvm-project?rev=247953&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=247953&view=rev</a><div><br></div><div>It has broken TestCppValueCast on Windows and fixed TestCxxWcharT on Windows as well.  Although these appear to be platform specific behavioral changes, as I don't see anyone else reporting failures or fixes, the description of the CL leads me to infer that this change was supposed to have no functional change.  I haven't gotten any response pinging the thread so I'll pose it to the wider list for visibility in case someone can shed some light on this.  In particular, is this supposed to come with a functional change, and if so what?</div><div><br></div><div>Unfortunately I don't have buildbots running tests on Windows yet, so I can't justify reverting the CL (and it's too old to revert now anyway since other things depend on it), but it would greatly appreciate if someone can take a look at this and see if they can identify the source of the behavioral change</div><div><br></div><div>I do plan to have a buildbot running tests within a week or two, so hopefully we can catch this kind of thing much sooner next time.</div></div></blockquote></div></div>