[PATCH] D129599: Apply PGO on SimpleLoopUnswitch

[Arthur Eubanks via Phabricator via llvm-commits]

aeubanks added inline comments.


================
Comment at: llvm/test/Transforms/SimpleLoopUnswitch/PGO-nontrivial-unswitch.ll:2
+
+; RUN: opt < %s -passes='require<profile-summary>,function(loop-mssa(simple-loop-unswitch<nontrivial>)),print<loops>' \
+; RUN:     --disable-output 2>&1 | sort -b -k 1 | FileCheck %s
----------------
fhahn wrote:
> aeubanks wrote:
> > aeubanks wrote:
> > > I'd just use `llvm/utils/update_test_checks.py` rather than manually checking loops
> > > once the test looks good it should be precommitted so we can see the delta this patch gives
> > the autogenerated CHECKs aren't in here, it should be
> > `; RUN: opt < %s -passes='require<profile-summary>,function(loop-mssa(simple-loop-unswitch<nontrivial>)),print<loops>' -S | FileCheck %`, then delete the existing CHECKs and run `update_test_checks.py`
> `,print<loops>'` is this needed? It looks like you are not checking the output (which is emitted to stderr)
sorry, typo, yeah we don't need `print<loops>`


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D129599/new/

https://reviews.llvm.org/D129599