This is an archive of the discontinued LLVM Phabricator instance.

[scudo][standalone] Only use yield on ARMv6K and newer
Needs ReviewPublic

Authored by benwolsieffer on Dec 7 2022, 7:35 PM.
This revision needs review, but there are no reviewers specified.

Details

Reviewers
None
Summary

The yield instruction is only available in ARMv6K and newer. It behaves as a
nop on single threaded platforms anyway, so use nop instead on unsupported
architectures.

Diff Detail

Event Timeline

benwolsieffer created this revision.Dec 7 2022, 7:35 PM
Herald added a project: Restricted Project. · View Herald TranscriptDec 7 2022, 7:35 PM
benwolsieffer requested review of this revision.Dec 7 2022, 7:35 PM
Herald added a project: Restricted Project. · View Herald TranscriptDec 7 2022, 7:35 PM
Herald added a subscriber: Restricted Project. · View Herald Transcript

Rebase on main

Chia-hungDuan added inline comments.Aug 28 2023, 10:13 PM
compiler-rt/lib/scudo/standalone/common.h
104–120

FYI, this will be deprecated in D156951