[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