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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to