This is an archive of the discontinued LLVM Phabricator instance.

[compiler-rt][bultins] Fix libatomic standalone build
ClosedPublic

Authored by abrachet on May 29 2023, 10:12 PM.

Diff Detail

Event Timeline

abrachet created this revision.May 29 2023, 10:12 PM
Herald added a project: Restricted Project. · View Herald TranscriptMay 29 2023, 10:12 PM
Herald added subscribers: Enna1, dberris. · View Herald Transcript
abrachet requested review of this revision.May 29 2023, 10:12 PM
phosek accepted this revision.May 29 2023, 11:37 PM

LGTM

compiler-rt/lib/builtins/CMakeLists.txt
869–870

This can be removed.

883

Since this will only ever be set to builtins, can we just hardcode it here?

This revision is now accepted and ready to land.May 29 2023, 11:37 PM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptMay 30 2023, 11:19 AM
Herald added a subscriber: Restricted Project. · View Herald Transcript
abrachet marked 2 inline comments as done.May 30 2023, 11:20 AM

Comments addressed in commit