[PATCH] D44555: clang-interpreter example cmake fix

Luke Cheeseman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 16 03:43:01 PDT 2018


LukeCheeseman created this revision.
LukeCheeseman added a reviewer: marsupial.
Herald added subscribers: cfe-commits, mgorny.

Add in a space when appending the export to the linker options. Without
the space the export is appeneded onto whatever the last link option
was, which might be a file.


Repository:
  rC Clang

https://reviews.llvm.org/D44555

Files:
  examples/clang-interpreter/CMakeLists.txt


Index: examples/clang-interpreter/CMakeLists.txt
===================================================================
--- examples/clang-interpreter/CMakeLists.txt
+++ examples/clang-interpreter/CMakeLists.txt
@@ -34,7 +34,7 @@
   # Is this a CMake bug that even with export_executable_symbols, Windows
   # needs to explictly export the type_info vtable
   set_property(TARGET clang-interpreter
-               APPEND_STRING PROPERTY LINK_FLAGS /EXPORT:??_7type_info@@6B@)
+               APPEND_STRING PROPERTY LINK_FLAGS " /EXPORT:??_7type_info@@6B@")
 endif()
 
 function(clang_enable_exceptions TARGET)


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D44555.138677.patch
Type: text/x-patch
Size: 607 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180316/4194a791/attachment-0001.bin>


More information about the cfe-commits mailing list