[llvm] r227479 - Disable compilation of llvm-pdbdump for versions of MSVC < 2013.
zturner at google.com
Thu Jan 29 19:39:58 PST 2015
That's strange. I don't know enough about CMake to understand why this was
necessary, but if it works for you, can you commit that fix? If this is
still causing an issue tomorrow, I will just work on a proper fix for VS
On Thu Jan 29 2015 at 5:16:21 PM Kaylor, Andrew <andrew.kaylor at intel.com>
> This still didn't fix it for me.
> I don't know if this is quirk of the version of cmake I'm using, but I
> needed to do this to get it to not try to build llvm-pdbdump:
> Index: tools/CMakeLists.txt
> --- tools/CMakeLists.txt (revision 227508)
> +++ tools/CMakeLists.txt (working copy)
> @@ -66,6 +66,8 @@
> # MSVC 2013 and higher. Since this is strictly a utility, and since we
> # to drop support for MSVC 2012 soon, don't build this for MSVC < 2013.
> + ignore_llvm_tool_subdirectory(llvm-pdbdump)
> if(NOT CYGWIN AND LLVM_ENABLE_PIC)
> -----Original Message-----
> From: llvm-commits-bounces at cs.uiuc.edu [mailto:llvm-commits-bounces@
> cs.uiuc.edu] On Behalf Of Zachary Turner
> Sent: Thursday, January 29, 2015 10:44 AM
> To: llvm-commits at cs.uiuc.edu
> Subject: [llvm] r227479 - Disable compilation of llvm-pdbdump for versions
> of MSVC < 2013.
> Author: zturner
> Date: Thu Jan 29 12:44:14 2015
> New Revision: 227479
> URL: http://llvm.org/viewvc/llvm-project?rev=227479&view=rev
> Disable compilation of llvm-pdbdump for versions of MSVC < 2013.
> Certain aspects of llvm-pdbdump require language support only present in
> MSVC 2013 and higher. Since this is strictly a utility, and since we hope
> to drop support for MSVC 2012 soon, don't build this unless MSVC 2013 or
> Modified: llvm/trunk/tools/CMakeLists.txt
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/
> --- llvm/trunk/tools/CMakeLists.txt (original)
> +++ llvm/trunk/tools/CMakeLists.txt Thu Jan 29 12:44:14 2015
> @@ -61,7 +61,10 @@ add_llvm_tool_subdirectory(yaml2obj)
> +if(MSVC AND NOT(MSVC_VERSION LESS 1800))
> + # Certain aspects of llvm-pdbdump require language support only
> +present in
> + # MSVC 2013 and higher. Since this is strictly a utility, and since
> +we hope
> + # to drop support for MSVC 2012 soon, don't build this for MSVC < 2013.
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the llvm-commits