[PATCH] D119486: [docs] Replace `opt -analyze` with better alternatives.
Alina Sbirlea via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Feb 10 15:37:05 PST 2022
asbirlea accepted this revision.
asbirlea added a comment.
This revision is now accepted and ready to land.
lgtm.
================
Comment at: llvm/utils/update_analyze_test_checks.py:17
3. Update the RUN-lines in the affected regression tests to look canonical.
- Example: "; RUN: opt < %s -analyze -cost-model -S | FileCheck %s"
+ Example: "; RUN: opt < %s -passes='print<cost-model>' -S 2>&1 -disable-output | FileCheck %s"
4. Refresh the FileCheck lines for either the entire file or select functions by
----------------
nit: unnecessary `-S` with `-disable-output`?
Perhaps update to:
`; RUN: opt < %s -passes='print<cost-model>' -disable-output 2>&1 | FileCheck %s"`
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D119486/new/
https://reviews.llvm.org/D119486
More information about the llvm-commits
mailing list