This is an archive of the discontinued LLVM Phabricator instance.

Adapt `tsan/flush_memory.cpp` to run on non-local platforms.
ClosedPublic

Authored by delcypher on Sep 29 2021, 10:32 AM.

Details

Summary

ad890aa2327feb6b6aee676fe85b2352fba2403e landed a test without
using the %run prefix which means the test fails to run for
platforms that need it (e.g. iOS simulators).

This patch adds the %run prefix. While we're here also split
the single RUN line into two to make debugging easier.

rdar://83637296

Diff Detail

Event Timeline

delcypher requested review of this revision.Sep 29 2021, 10:32 AM
delcypher created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptSep 29 2021, 10:32 AM
Herald added a subscriber: Restricted Project. · View Herald Transcript
dvyukov accepted this revision.Sep 29 2021, 10:34 AM
This revision is now accepted and ready to land.Sep 29 2021, 10:34 AM

@dvyukov Thanks for the review.

This revision was landed with ongoing or failed builds.Sep 29 2021, 10:39 AM
This revision was automatically updated to reflect the committed changes.