This is an archive of the discontinued LLVM Phabricator instance.

[runtimes] Move common functions from Handle{Libcxx,Libcxxabi,Libunwind}Flags.cmake to runtimes/cmake/Modules/HandleFlags.cmake
ClosedPublic

Authored by philnik on Feb 20 2023, 7:34 AM.

Details

Reviewers
ldionne
phosek
Group Reviewers
Restricted Project
Restricted Project
Restricted Project
Commits
rG0af67d167d6c: [runtimes] Move common functions from Handle{Libcxx,Libcxxabi,Libunwind}Flags.

Diff Detail

Event Timeline

philnik created this revision.Feb 20 2023, 7:34 AM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptFeb 20 2023, 7:34 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript
philnik requested review of this revision.Feb 20 2023, 7:34 AM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptFeb 20 2023, 7:34 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript
Herald added a reviewer: Restricted Project. · View Herald Transcript
arichardson added inline comments.
runtimes/cmake/Modules/HandleFlags.cmake
45

Is there a reason that this appends twice?

philnik added inline comments.Feb 20 2023, 1:08 PM
runtimes/cmake/Modules/HandleFlags.cmake
45

No idea, but it's been that way already.

phosek accepted this revision.Feb 20 2023, 2:21 PM

LGTM

libunwind/cmake/Modules/HandleLibunwindFlags.cmake
280–283

This could be removed as well.

philnik accepted this revision as: Restricted Project.Feb 21 2023, 4:14 AM
philnik marked an inline comment as done.
This revision is now accepted and ready to land.Feb 21 2023, 4:14 AM
This revision was landed with ongoing or failed builds.Feb 21 2023, 4:15 AM
This revision was automatically updated to reflect the committed changes.