This is an archive of the discontinued LLVM Phabricator instance.

[libc++][test] Makes pthread a flag.
ClosedPublic

Authored by Mordante on Apr 28 2023, 8:37 AM.

Details

Reviewers
ldionne
Group Reviewers
Restricted Project
Commits
rGd58ba7a4d145: [libc++][test] Makes pthread a flag.
Summary

Using pthread may affect the compilation too not only the linking.
Therefore pthread is changed from a link flag to a generic flag used for
both compilation and linking.

This fixes an issue when using modules since the use of the pthread flag
makes the BMI files not compatible.

Diff Detail

Event Timeline

Mordante created this revision.Apr 28 2023, 8:37 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 28 2023, 8:37 AM
Mordante requested review of this revision.Apr 28 2023, 8:37 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 28 2023, 8:37 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript
Mordante updated this revision to Diff 518469.May 1 2023, 9:18 AM

CI fixes.

Mordante updated this revision to Diff 518758.May 2 2023, 8:43 AM

CI fixes.

ldionne accepted this revision.May 3 2023, 9:01 AM
This revision is now accepted and ready to land.May 3 2023, 9:01 AM
This revision was landed with ongoing or failed builds.May 3 2023, 9:35 AM
This revision was automatically updated to reflect the committed changes.