[PATCH] D44245: Propagate flags to SDValue in SplitVecOp_VECREDUCE

Amara Emerson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 8 14:32:54 PST 2018


aemerson accepted this revision.
aemerson added a comment.
This revision is now accepted and ready to land.

LGTM, can you also make the test name a little more specific, e.g. 'vecreduce-propagate-sd-flags.ll'?



================
Comment at: test/CodeGen/AArch64/propagate-sdvalue-flags.ll:1
+; REQUIRES: arm-registered-target
+; RUN: llc %s -debug-only=legalize-types 2>&1 | FileCheck %s
----------------
You will also need a `REQUIRES: asserts` since you use the -debug-only option.



https://reviews.llvm.org/D44245





More information about the llvm-commits mailing list