[PATCH] D147394: [NewPM] Use PassID instead of pass name

Arthur Eubanks via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 13 15:07:25 PDT 2023


aeubanks added a comment.

some test nits, otherwise lgtm
do you need me to land this for you?



================
Comment at: llvm/test/Other/loop-print-after-pass-invalidated.ll:2
+; RUN: opt < %s 2>&1 -disable-output \
+; RUN: 	   -passes='loop(simple-loop-unswitch)' -enable-nontrivial-unswitch \
+; RUN:     -print-after=simple-loop-unswitch \
----------------



================
Comment at: llvm/test/Other/loop-print-after-pass-invalidated.ll:4
+; RUN:     -print-after=simple-loop-unswitch \
+; RUN:	   | FileCheck %s -check-prefix=DEL
+
----------------
there's no `DEL` check prefix in the test?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147394



More information about the llvm-commits mailing list