[PATCH] D28307: Add Instruction number to LSR cost model (PR23384) part 1 of 3

[Hal Finkel via Phabricator via llvm-commits]

hfinkel added inline comments.


================
Comment at: test/Transforms/LoopStrengthReduce/X86/lsr-insns-1.ll:14
+
+; CHECK: 7 asm-printer      - Number of machine instrs printed
+
----------------
Ah, okay. You're just counting the total number of instructions. Please actually match the desired output instruction pattern. Otherwise these tests will be fragile to unrelated changes.


Repository:
  rL LLVM

https://reviews.llvm.org/D28307