[PATCH] D60854: [DAGLegalize][PowerPC] Add promote legalization of addc/adde and subc/sube
Eli Friedman via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Apr 25 12:41:30 PDT 2019
efriedma requested changes to this revision.
efriedma added inline comments.
This revision now requires changes to proceed.
================
Comment at: llvm/test/CodeGen/PowerPC/pr40922.ll:1
+; RUN: llc -verify-machineinstrs -mtriple=powerpc-unknown-linux-gnu < %s
+
----------------
Err, sorry, I didn't read the test carefully enough.
Please use FileCheck to add some CHECK lines; even though the original issue was just a crash, adding a few lines to check for the expected addc or whatever might catch other issues in the future.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D60854/new/
https://reviews.llvm.org/D60854
More information about the llvm-commits
mailing list