[PATCH] D58184: [scudo][standalone] Introduce platform specific code & mutexes

Brian Rzycki via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 28 11:05:56 PST 2019


brzycki added a comment.

In D58184#1413905 <https://reviews.llvm.org/D58184#1413905>, @cryptoad wrote:

> I seemed to have fixed it locally:
>
>   add_library("RTScudoStandalone.test.${arch}" STATIC
>     $<TARGET_OBJECTS:RTScudoStandalone.${arch}>)
>   
>
> The STATIC was missing. Let me know if that works for you.


Yes, that worked for me as well on Ubuntu. Thank you! I was deep in the bowels of recursive CMake macros when I noticed your update. I will also try this on RHEL 6.7 to see if it fixes it for `clock_gettime()`. I know you don't have that environment so I'll have to figure that one out if it doesn't work.


Repository:
  rCRT Compiler Runtime

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

https://reviews.llvm.org/D58184





More information about the llvm-commits mailing list