This is an archive of the discontinued LLVM Phabricator instance.

[ARM] Add documentation for -march= and -mfpu= command line options
AcceptedPublic

Authored by chill on Jan 27 2020, 3:25 AM.

Diff Detail

Event Timeline

chill created this revision.Jan 27 2020, 3:25 AM
efriedma accepted this revision.Jan 27 2020, 4:38 PM

LGTM; thanks.

This revision is now accepted and ready to land.Jan 27 2020, 4:38 PM
rovka added a subscriber: rovka.Jan 28 2020, 12:48 AM

Just a few nits.

clang/docs/ClangCommandLineReference.rst
2467

Typo: can

2468

I think a full sentence with an example would be more helpful than just this parenthesis.

2469

This could also use an example, otherwise it's a bit obscure.

2526

Missing punctuation.

chill updated this revision to Diff 240854.Jan 28 2020, 6:11 AM
chill retitled this revision from [ARM] Add documentation for -march= and -mfpu= copmmand line options to [ARM] Add documentation for -march= and -mfpu= command line options.
chill marked 4 inline comments as done.
chill added a reviewer: rovka.
rovka accepted this revision.Jan 29 2020, 12:38 AM

Thanks, LGTM!

This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptJan 29 2020, 2:40 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript
efriedma reopened this revision.Jun 7 2023, 1:30 PM

This patch was reverted by 9624beb38a46037f69362650a52e06d8be4fd006 . I have a patch planned for some similar accidental reverts, but this seems non-trivial to pull forward, so I'm not going to try to address it now.

This revision is now accepted and ready to land.Jun 7 2023, 1:30 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 7 2023, 1:30 PM