[PATCH] D94853: [NFC] Disallow unused prefixes under Other

Jinsong Ji via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 21 13:08:53 PST 2021


jsji added inline comments.


================
Comment at: llvm/test/Other/print-slotindexes.ll:1
-; RUN: llc -print-after=slotindexes < %s 2>&1 | FileCheck %s --check-prefixes=ALL,SI 
-; RUN: llc -print-after=slotindexes -print-slotindexes=false < %s 2>&1 | FileCheck %s --check-prefixes=ALL,NOSI
+; RUN: llc -print-after=slotindexes < %s 2>&1 | FileCheck %s --check-prefixes=CHECK,SI 
+; RUN: llc -print-after=slotindexes -print-slotindexes=false < %s 2>&1 | FileCheck %s --check-prefixes=CHECK,NOSI
----------------
mtrofin wrote:
> @jsji : I think s/ALL/CHECK was the intention, correct? (The test passes)
Yes, you are right. Thanks for fixing.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94853



More information about the llvm-commits mailing list