LGTM<br><br><div class="gmail_quote"><div dir="ltr">On Thu, May 10, 2018, 1:25 PM JF Bastien via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">jfb created this revision.<br>
Herald added subscribers: aheejin, mgorny.<br>
<br>
As discussed in: <a href="http://lists.llvm.org/pipermail/llvm-dev/2018-May/123182.html" rel="noreferrer" target="_blank">http://lists.llvm.org/pipermail/llvm-dev/2018-May/123182.html</a><br>
<br>
Let's start migrating to C++14. As a first step, update the GCC and LLVM<br>
versions to the ones that first got full C++14 support, and see if bots fall<br>
over and die. If that sticks, let's then update the rest of this file (MSVC and<br>
standard libraries). If it all works out (let's be honest: it won't for a while)<br>
then we can bump to -std=c++14.<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D46723" rel="noreferrer" target="_blank">https://reviews.llvm.org/D46723</a><br>
<br>
Files:<br>
  cmake/modules/CheckCompilerVersion.cmake<br>
<br>
<br>
Index: cmake/modules/CheckCompilerVersion.cmake<br>
===================================================================<br>
--- cmake/modules/CheckCompilerVersion.cmake<br>
+++ cmake/modules/CheckCompilerVersion.cmake<br>
@@ -1,19 +1,19 @@<br>
-# Check if the host compiler is new enough. LLVM requires at least GCC 4.8,<br>
-# MSVC 2015 (Update 3), or Clang 3.1.<br>
+# Check if the host compiler is new enough. LLVM requires at least GCC 5.0,<br>
+# MSVC 2015 (Update 3), or Clang 3.4.<br>
<br>
 include(CheckCXXSourceCompiles)<br>
<br>
 if(NOT DEFINED LLVM_COMPILER_CHECKED)<br>
   set(LLVM_COMPILER_CHECKED ON)<br>
<br>
   if(NOT LLVM_FORCE_USE_OLD_TOOLCHAIN)<br>
     if(CMAKE_CXX_COMPILER_ID STREQUAL "GNU")<br>
-      if(CMAKE_CXX_COMPILER_VERSION VERSION_LESS 4.8)<br>
-        message(FATAL_ERROR "Host GCC version must be at least 4.8!")<br>
+      if(CMAKE_CXX_COMPILER_VERSION VERSION_LESS 5.0)<br>
+        message(FATAL_ERROR "Host GCC version must be at least 5.0!")<br>
       endif()<br>
     elseif(CMAKE_CXX_COMPILER_ID STREQUAL "Clang")<br>
-      if(CMAKE_CXX_COMPILER_VERSION VERSION_LESS 3.1)<br>
-        message(FATAL_ERROR "Host Clang version must be at least 3.1!")<br>
+      if(CMAKE_CXX_COMPILER_VERSION VERSION_LESS 3.4)<br>
+        message(FATAL_ERROR "Host Clang version must be at least 3.4!")<br>
       endif()<br>
<br>
       if (CMAKE_CXX_SIMULATE_ID MATCHES "MSVC")<br>
<br>
<br>
</blockquote></div>