This is an archive of the discontinued LLVM Phabricator instance.

[flang][nfc] Update README.md
ClosedPublic

Authored by awarzynski on Jan 12 2022, 2:48 AM.

Details

Summary

With https://reviews.llvm.org/D116731 merged, installing Clang, MLIR or
LLVM is no longer required for standalone builds. For consistency sake,
remove "installation" from the build instrucitons.

Diff Detail

Event Timeline

awarzynski created this revision.Jan 12 2022, 2:48 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 12 2022, 2:48 AM
awarzynski requested review of this revision.Jan 12 2022, 2:48 AM
PeteSteinfeld added inline comments.Jan 12 2022, 6:37 AM
flang/README.md
39–46

This section states that it's necessary to create the install area for a later standalone build to succeed. But this is no longer needed.

awarzynski added inline comments.Jan 12 2022, 11:49 AM
flang/README.md
39–46

Thanks for pointing this out!

Update the "Building flang" section, as per Pete's suggestion.

PeteSteinfeld accepted this revision.Jan 12 2022, 11:52 AM

Looks great!

Thanks for doing this, Andrzej. You rock.

This revision is now accepted and ready to land.Jan 12 2022, 11:52 AM

Thanks for doing this, Andrzej. You rock.

Thank you, Pete, that's very kind of you! Users like you make it worthwhile :)

This revision was automatically updated to reflect the committed changes.