[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