[PATCH] D144562: [NFC] Refine tests by adding `:` to checks

Mariya Podchishchaeva via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 22 07:39:37 PST 2023


Fznamznon added inline comments.


================
Comment at: llvm/test/MC/Hexagon/multiple-pc4.s:1
-# RUN: llvm-mc -arch=hexagon -filetype=asm %s 2>%t; FileCheck --implicit-check-not=error %s <%t
+# RUN: llvm-mc -arch=hexagon -filetype=asm %s 2>%t; FileCheck --implicit-check-not=error: %s <%t
 
----------------
jhenderson wrote:
> Not related to your change, but does this line actually run FileCheck at all? I'm pretty sure it's missing a `|` character somewhere... Same goes with a few other tests you've modified.
> 
> (I am not familiar with these tests though, so my comment may be incorrect).
I think it runs FileCheck. In this patch I modified only tests that were complaining about "error" substring in the path.
I think the magic is hidden in combination of "2>%t" in llvm-mc command and following "<%t" in FileCheck command.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144562



More information about the llvm-commits mailing list