This is an archive of the discontinued LLVM Phabricator instance.

Fix xray fdr mode to allow multiple flushes
ClosedPublic

Authored by toddlipcon on Feb 9 2021, 4:31 PM.

Diff Detail

Event Timeline

toddlipcon created this revision.Feb 9 2021, 4:31 PM
toddlipcon requested review of this revision.Feb 9 2021, 4:31 PM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 9 2021, 4:31 PM
Herald added a subscriber: Restricted Project. · View Herald Transcript
dberris accepted this revision.Feb 9 2021, 4:46 PM

LGTM

Thanks!

This revision is now accepted and ready to land.Feb 9 2021, 4:46 PM
MaskRay added a subscriber: MaskRay.Feb 9 2021, 5:16 PM
MaskRay added inline comments.
compiler-rt/test/xray/TestCases/Posix/fdr-reinit.cpp
52–55

Worth a comment why the loop exists.

This revision was landed with ongoing or failed builds.Feb 9 2021, 5:57 PM
This revision was automatically updated to reflect the committed changes.