[PATCH] D28307: Add Instruction number to LSR cost model (PR23384) part 1 of 3
Quentin Colombet via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jan 31 14:27:39 PST 2017
qcolombet accepted this revision.
qcolombet added a comment.
This revision is now accepted and ready to land.
LGTM, nitpicks on the check pattern for the llc test case.
================
Comment at: test/Transforms/LoopStrengthReduce/X86/lsr-insns-1.ll:29
+; CHECK: jne
+
+target datalayout = "e-m:e-i64:64-f80:128-n8:16:32:64-S128"
----------------
I usually like to check that the operands are correct as well.
Repository:
rL LLVM
https://reviews.llvm.org/D28307
More information about the llvm-commits
mailing list