[libc-commits] [PATCH] D106919: [libc][wip] add integration tests for scudo in libc
Michael Jones via Phabricator via libc-commits
libc-commits at lists.llvm.org
Mon Aug 2 16:03:36 PDT 2021
michaelrj added inline comments.
Comment at: compiler-rt/cmake/config-ix.cmake:703
+ set(SCUDO_STANDALONE_BUILD_OBJECTS TRUE)
> why create a copy of `SCUDO_STANDALONE_SUPPORTED_ARCH`?
I figured it was probably better to condition off of a proper bool as opposed to a list, but if we're not using this at all then I can remove it.
Comment at: compiler-rt/lib/gwp_asan/CMakeLists.txt:79-81
> can revert this diff now that COMPILER_RT_HAS_GWP_ASAN is back to being the source of truth
`COMPILER_RT_HAS_GWP_ASAN` being the source of truth is useful, but from what I can tell it's still conditioned on the compiler-rt supported architectures. The problem there is that I don't see an easy way to tell compiler-rt that you want to compile for an architecture that isn't the current one. If there's a cmake flag to do that that I have missed, then that would be very helpful.
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the libc-commits