[PATCH] D71786: RFC: [Support] On Windows, add optional support for rpmalloc

[Russell Gallop via Phabricator via llvm-commits]

russell.gallop added a comment.

Thanks for the patch. I applied it (to 8188c998ffa4d20253444b257402907d2aa74dc2 <https://reviews.llvm.org/rG8188c998ffa4d20253444b257402907d2aa74dc2>), specified LLVM_ENABLE_RPMALLOC=ON and -DLLVM_USE_CRT_RELEASE=MT and tried to build with MS cl.exe/link.exe. It fails creating a library:

     Creating library lib\LLVM-C.lib and object lib\LLVM-C.exp
  LLVMSupport.lib(rpmalloc.c.obj) : error LNK2005: calloc already defined in libucrt.lib(calloc.obj)
  LLVMSupport.lib(rpmalloc.c.obj) : error LNK2005: free already defined in libucrt.lib(free.obj)
  LLVMSupport.lib(rpmalloc.c.obj) : error LNK2005: malloc already defined in libucrt.lib(malloc.obj)
  bin\LLVM-C.dll : fatal error LNK1169: one or more multiply defined symbols found

Is there any reason this couldn't be built with these host tools? If not, the please could you get this working? Thanks.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D71786/new/

https://reviews.llvm.org/D71786