This is an archive of the discontinued LLVM Phabricator instance.

[docs] Replace `opt -analyze` with better alternatives.
ClosedPublic

Authored by aeubanks on Feb 10 2022, 2:25 PM.

Details

Summary

opt -analyze is legacy PM-specific. Show better ways of doing the same
thing, generally with some sort of -passes=print<foo>.

Diff Detail

Event Timeline

aeubanks created this revision.Feb 10 2022, 2:25 PM
aeubanks requested review of this revision.Feb 10 2022, 2:25 PM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 10 2022, 2:25 PM
asbirlea accepted this revision.Feb 10 2022, 3:37 PM

lgtm.

llvm/utils/update_analyze_test_checks.py
17

nit: unnecessary -S with -disable-output?
Perhaps update to:
; RUN: opt < %s -passes='print<cost-model>' -disable-output 2>&1 | FileCheck %s"

This revision is now accepted and ready to land.Feb 10 2022, 3:37 PM
aeubanks marked an inline comment as done.Feb 10 2022, 3:39 PM
This revision was landed with ongoing or failed builds.Feb 10 2022, 3:39 PM
This revision was automatically updated to reflect the committed changes.