[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