[PATCH] D92935: Introduce support for PowerPC devices with an Embedded Floating-point APU version 2 (efpu2)
Jessica Clarke via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Jul 28 07:04:27 PDT 2021
jrtc27 added inline comments.
================
Comment at: llvm/test/CodeGen/PowerPC/spe.ll:2
; NOTE: Assertions have been autogenerated by utils/update_llc_test_checks.py
-; RUN: llc -verify-machineinstrs < %s -mtriple=powerpc-unknown-linux-gnu \
-; RUN: -mattr=+spe | FileCheck %s
+; RUN: split-file %s %t
+; RUN: llc -verify-machineinstrs < %t/single.ll -mtriple=powerpc-unknown-linux-gnu \
----------------
jrtc27 wrote:
> This breaks being able to run update_llc_test_checks.py without first running the test itself to generate the files
In fact it breaks it completely, update_llc_test_checks.py doesn't support the %t substitutions, and even if it did it's unclear what that even means; should it update the CHECK lines in the file with the RUN lines or the files being used by FileCheck?
This should have just been manually split.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D92935/new/
https://reviews.llvm.org/D92935
More information about the cfe-commits
mailing list