This is an archive of the discontinued LLVM Phabricator instance.

[openmp] Only try to add the -mrtm flag on i386/x86_64
ClosedPublic

Authored by mstorsjo on Nov 9 2022, 2:26 PM.

Details

Summary

Clang does accept the flag for any architecture, but prints a
warning:

clang-16: warning: argument unused during compilation: '-mrtm' [-Wunused-command-line-argument]

This flag is documented as an x86 specific flag, so don't try to
add it for other architectures, to silence this warning.

Diff Detail

Event Timeline

mstorsjo created this revision.Nov 9 2022, 2:26 PM
Herald added a project: Restricted Project. · View Herald TranscriptNov 9 2022, 2:26 PM
mstorsjo requested review of this revision.Nov 9 2022, 2:26 PM
Herald added a project: Restricted Project. · View Herald Transcript
Herald added a subscriber: sstefan1. · View Herald Transcript
This revision is now accepted and ready to land.Nov 17 2022, 2:19 PM
This revision was automatically updated to reflect the committed changes.