This is an archive of the discontinued LLVM Phabricator instance.

[SCUDO] add cmake options for custom sysroot
ClosedPublic

Authored by michaelrj on Oct 11 2022, 11:08 AM.

Details

Summary

These options will allow the SCUDO standalone to be built with custom
headers. Specifically, this patch will enable building with the
LLVM-libc headers.

Diff Detail

Event Timeline

michaelrj created this revision.Oct 11 2022, 11:08 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 11 2022, 11:08 AM
michaelrj requested review of this revision.Oct 11 2022, 11:08 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 11 2022, 11:08 AM
Herald added a subscriber: Restricted Project. · View Herald Transcript
This revision is now accepted and ready to land.Oct 11 2022, 11:45 AM
michaelrj updated this revision to Diff 466958.Oct 11 2022, 3:50 PM

add flag for easier building of scudo with the LLVM libc headers

abrachet accepted this revision.Oct 11 2022, 7:09 PM
abrachet added inline comments.
compiler-rt/lib/scudo/standalone/CMakeLists.txt
145

I think you can do include_directories(SYSTEM ${COMPILER_RT_BINARY_DIR}/../libc/include/) and then you won't need "-I ${COMPILER_RT_BINARY_DIR}/../libc/include/". But not sure

michaelrj marked an inline comment as done.

remove redundant cflag

compiler-rt/lib/scudo/standalone/CMakeLists.txt
145

ah, you're right. Fixed

This revision was automatically updated to reflect the committed changes.