[PATCH] D106969: [NFC][PowerPC] Fix spe.ll to work with update_llc_test_checks.py again

Jessica Clarke via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 28 08:17:43 PDT 2021


jrtc27 created this revision.
jrtc27 added reviewers: PowerPC, kiausch, nemanjai, jhibbits.
Herald added a subscriber: shchenz.
jrtc27 requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

Using split-file does not work with update_llc_test_checks.py. It's also
mostly redundant, as the single and double tests can just use a single
llc and FileCheck invocation for each FPU type using -check-prefixes
rather than -check-prefix, and update_llc_test_checks.py will merge what
it can. Only test_dasmconst needs to be SPE-only and so is pulled out
into its own mall file (rather than using sed to preprocess the file and
keep it commented out for EFPU2, which would work, but is ugly).

As well as cutting down on the number of RUN lines, this also results in
test_fma's CHECK lines being merged for both FPUs.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D106969

Files:
  llvm/test/CodeGen/PowerPC/spe-hwdouble.ll
  llvm/test/CodeGen/PowerPC/spe.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D106969.362399.patch
Type: text/x-patch
Size: 22586 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210728/ba54bb28/attachment.bin>


More information about the llvm-commits mailing list