<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
<meta name="Generator" content="Microsoft Word 14 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
{font-family:Tahoma;
panose-1:2 11 6 4 3 5 4 4 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0in;
margin-bottom:.0001pt;
font-size:12.0pt;
font-family:"Times New Roman","serif";}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:blue;
text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
{mso-style-priority:99;
color:purple;
text-decoration:underline;}
p.MsoAcetate, li.MsoAcetate, div.MsoAcetate
{mso-style-priority:99;
mso-style-link:"Balloon Text Char";
margin:0in;
margin-bottom:.0001pt;
font-size:8.0pt;
font-family:"Tahoma","sans-serif";}
span.BalloonTextChar
{mso-style-name:"Balloon Text Char";
mso-style-priority:99;
mso-style-link:"Balloon Text";
font-family:"Tahoma","sans-serif";}
span.EmailStyle19
{mso-style-type:personal-reply;
font-family:"Calibri","sans-serif";
color:#1F497D;}
.MsoChpDefault
{mso-style-type:export-only;
font-family:"Calibri","sans-serif";}
@page WordSection1
{size:8.5in 11.0in;
margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
{page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
</head>
<body lang="EN-US" link="blue" vlink="purple">
<div class="WordSection1">
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D">I have been seeing some problems where some clang regression tests intermittently fail -- most often Format/style-on-command-line. I have no idea if that’s
related.<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D">When I say I was able to build this morning, I mean that I was able to build before I updated this afternoon. My code base was from last Wednesday, so it may
be that something has changed since then that I also would have run into.<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D">I don’t know what to say about reverting this change. This change seems like the kind of thing that needs to happen sooner or later. If it stays, I’m going
to be forced to use a local change to enable builds until I can get the techs here to update a couple of systems for me. I can live with that, but it is inconvenient. Mainly I just wanted to start a conversation to see how many other people were in my position
and to explore the possibility that there are other ways around whatever issues forced this change.<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D">-Andy<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoNormal"><b><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif"">From:</span></b><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif""> filcab@gmail.com [mailto:filcab@gmail.com]
<b>On Behalf Of </b>Filipe Cabecinhas<br>
<b>Sent:</b> Tuesday, February 17, 2015 4:40 PM<br>
<b>To:</b> Kaylor, Andrew<br>
<b>Cc:</b> llvm-commits@cs.uiuc.edu<br>
<b>Subject:</b> Re: [llvm] r229584 - We require MSVC 2013 Update 4 due to previous versions miscompiling ASTMatchers<o:p></o:p></span></p>
<p class="MsoNormal"><o:p> </o:p></p>
<div>
<p class="MsoNormal">ASTMatchers tests broke over the weekend due to a silent miscompile, as far as we could understand.<o:p></o:p></p>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Could you run check-clang and have it work, before this commit?<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">When committing, I figured it was a small change, so there shouldn't be big problems, but after committing I figured there might be people who wouldn't be ready, and I hadn't given a heads up. I then figured I would wait and see if someone
complained, either way.<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">If you want to, I can revert this patch, but this might prompt people to try to chase weird MSVC miscompilations. If we revert it we should probably make it so that llvm/clang compile and run tests on earlier revisions though, and actually
support point-releases (and document which ones are supported), which doesn't seem feasible to me (we have enough “problems” supporting major releases, sometimes).<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Let me know if you still want me to revert it, though.<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal"> Filipe<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
<div>
<p class="MsoNormal">On Tue, Feb 17, 2015 at 4:29 PM, Kaylor, Andrew <<a href="mailto:andrew.kaylor@intel.com" target="_blank">andrew.kaylor@intel.com</a>> wrote:<o:p></o:p></p>
<p class="MsoNormal">This is a fairly painful change for me.<br>
<br>
I didn't really mind the idea of not being able to build with MSVC 2012 because it's three years old or so. MSVC 2013 Update 4, on the other hand, is more like 3 months old. Granted it's still readily available, but it hasn't made its way through the lab
systems here yet and I suspect I'm not alone in experiencing this.<br>
<br>
I'm not sure what the issue is. This morning I could build just fine. Now this afternoon, I can't because CMake thinks my compiler is too old.<br>
<br>
What is the issue with ASTMatchers on earlier versions of MSVC? Is it something that was just exposed by a recent commit? I hadn't noticed a problem.<br>
<br>
-Andy<br>
<br>
<br>
-----Original Message-----<br>
From: <a href="mailto:llvm-commits-bounces@cs.uiuc.edu">llvm-commits-bounces@cs.uiuc.edu</a> [mailto:<a href="mailto:llvm-commits-bounces@cs.uiuc.edu">llvm-commits-bounces@cs.uiuc.edu</a>] On Behalf Of Filipe Cabecinhas<br>
Sent: Tuesday, February 17, 2015 3:29 PM<br>
To: <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
Subject: [llvm] r229584 - We require MSVC 2013 Update 4 due to previous versions miscompiling ASTMatchers<br>
<br>
Author: filcab<br>
Date: Tue Feb 17 17:29:00 2015<br>
New Revision: 229584<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=229584&view=rev" target="_blank">
http://llvm.org/viewvc/llvm-project?rev=229584&view=rev</a><br>
Log:<br>
We require MSVC 2013 Update 4 due to previous versions miscompiling ASTMatchers<br>
<br>
Previous versions of MSVC 2013 would miscompile ASTMatchers (and/or their tests). Bump up the requirement and make sure we know about the minor revision.<br>
<br>
Minimum required version found by Michael Edwards!<br>
<br>
Modified:<br>
llvm/trunk/cmake/modules/HandleLLVMOptions.cmake<br>
<br>
Modified: llvm/trunk/cmake/modules/HandleLLVMOptions.cmake<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/cmake/modules/HandleLLVMOptions.cmake?rev=229584&r1=229583&r2=229584&view=diff" target="_blank">
http://llvm.org/viewvc/llvm-project/llvm/trunk/cmake/modules/HandleLLVMOptions.cmake?rev=229584&r1=229583&r2=229584&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/cmake/modules/HandleLLVMOptions.cmake (original)<br>
+++ llvm/trunk/cmake/modules/HandleLLVMOptions.cmake Tue Feb 17 17:29:00<br>
+++ 2015<o:p></o:p></p>
<div>
<div>
<p class="MsoNormal">@@ -41,8 +41,8 @@ int main() { return (float)x; }"<br>
set(CMAKE_REQUIRED_LIBRARIES ${OLD_CMAKE_REQUIRED_LIBRARIES})<br>
endif()<br>
elseif(CMAKE_CXX_COMPILER_ID MATCHES "MSVC")<br>
- if(CMAKE_CXX_COMPILER_VERSION VERSION_LESS 18.0)<br>
- message(FATAL_ERROR "Host Visual Studio must be at least 2013 (MSVC 18.0)")<br>
+ if(CMAKE_CXX_COMPILER_VERSION VERSION_LESS 18.0.31101)<br>
+ message(FATAL_ERROR "Host Visual Studio must be at least 2013<br>
+ Update 4 (MSVC 18.0.31101)")<br>
endif()<br>
endif()<br>
endif()<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><o:p></o:p></p>
</div>
</div>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
</div>
</div>
</body>
</html>