Since we are looking to remove the old Scudo, we have to have a .so for
parity purposes as some platforms use it.
I tested this on Fuchsia & Linux, not on Android though.
Paths
| Differential D98456
[scudo][standalone] Add shared library to makefile ClosedPublic Authored by cryptoad on Mar 11 2021, 2:44 PM.
Details Summary Since we are looking to remove the old Scudo, we have to have a .so for I tested this on Fuchsia & Linux, not on Android though.
Diff Detail
Event TimelineComment Actions LGTM w/ nit
This revision is now accepted and ready to land.Mar 11 2021, 4:21 PM This revision was landed with ongoing or failed builds.Mar 15 2021, 8:13 AM Closed by commit rG752f477d677b: [scudo][standalone] Add shared library to makefile (authored by cryptoad). · Explain Why This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 330668 compiler-rt/lib/scudo/standalone/CMakeLists.txt
|
Shouldn't this be -pthread to ldflags, not linking lpthread?
i.e.
append_list_if(COMPILER_RT_HAS_LIBPTHREAD -pthread SCUDO_LINK_FLAGS)