<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">+ Chandler<div class=""><br class=""></div><div class="">Ok… What’s going on here?</div><div class=""><br class=""></div><div class="">I see Takumi re-landed the patches, but there have been a lot of changes that came in yesterday that go against the plan for rolling this out.</div><div class=""><br class=""></div><div class="">The original plan (as proposed by Chandler) was to wait a week after landing the CMake change before updating the documentation and allowing new C++11 features. This gives adequate time for people who may have missed this thread to adjust.</div><div class=""><br class=""></div><div class="">We now have uses of new C++11 features that aren’t supported by VS2012 littered around, which makes backing this change out far less trivial.</div><div class=""><br class=""></div><div class="">What is the plan now?</div><div class=""><br class=""></div><div class="">-Chris</div><div class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Feb 16, 2015, at 8:27 AM, Benjamin Kramer <<a href="mailto:benny.kra@gmail.com" class="">benny.kra@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><blockquote type="cite" style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><br class="Apple-interchange-newline">On 16.02.2015, at 17:02, Chris Bieneman <<a href="mailto:beanz@apple.com" class="">beanz@apple.com</a>> wrote:<br class=""><br class="">Takumi,<br class=""><br class="">Can you please provide an update? We've been working on this change for months, if there are blocking issues we need to know what they are.<br class=""></blockquote><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">In case you didn't see it, the patch was relanded yesterday and Takumi's bots are updated. I also updated the docs yesterday and a ton of cruft that accumulated to support MSVC 2012 was removed, there's no going back now ;)</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">What's left is updating our CodingStandards to properly reflect the supported C++11 features, as we've seen with the template aliases Microsoft's docs can't be trusted without manual verification. There are also some MSVC 2012 bots left on<span class="Apple-converted-space"> </span></span><a href="http://lab.llvm.org/" style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class="">lab.llvm.org</a><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class=""><span class="Apple-converted-space"> </span>that fail at the cmake stage now, 'lldb-x86-windows-msvc' and 'lld-x86_64-win7'. Not sure what to do about them</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">- Ben</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><blockquote type="cite" style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><blockquote type="cite" class="">On Feb 13, 2015, at 4:40 PM, NAKAMURA Takumi <<a href="mailto:geek4civic@gmail.com" class="">geek4civic@gmail.com</a>> wrote:<br class=""><br class="">Please wait. I'll revert it for now. I'll reland it when I am ready.<br class=""><br class="">2015-02-14 8:27 GMT+09:00 Chris Bieneman <<a href="mailto:beanz@apple.com" class="">beanz@apple.com</a>>:<br class=""><blockquote type="cite" class="">I have moved onto the next phase and committed r229185, which makes VS2013 our minimum version. I will revert if issues arise, and we can rinse and repeat as necessary.<br class=""><br class="">Once it sticks for a bit I’ll update the docs too.<br class=""><br class="">-Chris<br class=""><br class=""><blockquote type="cite" class="">On Feb 9, 2015, at 10:07 AM, Chris Bieneman <<a href="mailto:beanz@apple.com" class="">beanz@apple.com</a>> wrote:<br class=""><br class="">I agree with Aaron, this should not be a blocker because the workaround is simple. I'm also not opposed to raising the minimum CMake version all the way to 3.0. CMake 3 is 6 months old and CMake 3.1 is already out.<br class=""><br class="">I think most (if not all) our internal users are on 3.0 now, and those that may not be are either on old branches of LLVM or have no excuse not to update. I can start a separate thread with that proposal.<br class=""><br class="">-Chris<br class=""><br class="">Sent from my iPad<br class=""><br class=""><blockquote type="cite" class=""><blockquote type="cite" class="">On Feb 9, 2015, at 7:44 AM, Aaron Ballman <<a href="mailto:aaron@aaronballman.com" class="">aaron@aaronballman.com</a>> wrote:<br class=""><br class="">On Mon, Feb 9, 2015 at 8:36 AM, Greg Bedwell <<a href="mailto:gregbedwell@gmail.com" class="">gregbedwell@gmail.com</a>> wrote:<br class="">We've just encountered an issue with ninja and VS2013 when using versions of<br class="">CMake prior to 2.8.12.1. This isn't a combination that we typically use so<br class="">we've not run into it previously in our own builds. It isn't specifically<br class="">tied to upgrading the minimum version as it's a problem that already exists<br class="">but I figure we'll hit it sooner or later on one of the bots as people start<br class="">to use 2013 so best to deal with it sooner. Essentially, when building any<br class="">configuration through ninja with debug data enabled with anything more than<br class="">-j1 we'll hit an error like:<br class=""><br class="">C:\work\public_svn\llvm\lib\Support\APInt.cpp : fatal error C1041: cannot<br class="">open program database<br class="">'c:\work\public_svn\build-vs2013-native-ninja\lib\llvmsupport.pdb'; if<br class="">multiple CL.EXE write to the same .PDB file, please use /FS<br class=""><br class="">Here's the relevant CMake tracker which shows the fix going into 2.8.12.1:<br class=""><a href="http://www.cmake.org/Bug/view.php?id=14492" class="">http://www.cmake.org/Bug/view.php?id=14492</a><br class=""><br class="">I think our options are to either require CMake 2.8.12.1 for VS2013/ninja<br class="">builds (but I don't know how feasible this is) or we can put our own<br class="">workaround into our CMakelists to push the /FS switch when using Ninja and<br class="">Visual Studio, unless someone has any better ideas?<br class=""><br class="">Any thoughts? Is this a blocker for updating the minimum version?<br class=""></blockquote><br class="">I'm not certain I see it as a blocker since upgrading CMake should not<br class="">be too difficult (2.8.12.1 was released in Nov 2013). Whether we want<br class="">to require 2.8.12.1 is a different ball of wax, but for just the Ninja<br class="">+ MSVC build, I don't think that's too onerous (then again, I don't<br class="">use Ninja for my builds, either).<br class=""><br class="">~Aaron<br class=""></blockquote><br class="">_______________________________________________<br class="">LLVM Developers mailing list<br class=""><a href="mailto:LLVMdev@cs.uiuc.edu" class="">LLVMdev@cs.uiuc.edu</a> <a href="http://llvm.cs.uiuc.edu" class="">http://llvm.cs.uiuc.edu</a><br class=""><a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" class="">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br class=""></blockquote><br class=""><br class="">_______________________________________________<br class="">lldb-dev mailing list<br class=""><a href="mailto:lldb-dev@cs.uiuc.edu" class="">lldb-dev@cs.uiuc.edu</a><br class="">http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev<br class=""></blockquote></blockquote><br class="">_______________________________________________<br class="">lldb-dev mailing list<br class=""><a href="mailto:lldb-dev@cs.uiuc.edu" class="">lldb-dev@cs.uiuc.edu</a><br class=""><a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev" class="">http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev</a></blockquote></div></blockquote></div><br class=""></div></body></html>