[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.



More information about the llvm-commits mailing list