[PATCH] D26855: New unsafe-fp-math implementation for X86 target

[Andrew V. Tischenko via Phabricator via llvm-commits]

avt77 added inline comments.


================
Comment at: lib/Target/X86/X86InstrInfo.cpp:9523
+///
+///   vmovss  xmm1, dword ptr [rip + .LCPI0_0] # xmm1 = mem[0],zero,zero,zero
+///   vrcpss  xmm2, xmm0, xmm0
----------------
RKSimon wrote:
> avt77 wrote:
> > Gerolf wrote:
> > > input to this function are 7  parameters, the comment only lists 6.
> > I did not understand this comment: what should I do here?
> I think it means that while ArrayRef<int> Instrs have 7 instructions listed, the codegen in the comment only shows 6 instructions
But in fact all 7 instructions are shown but from index 0 to index 6 (maybe in "strange" order: 0,2,1,3,4,5,6). If you'd like I could change the order and/or start numbering from 1. Gerolf, should I do it or we fixed everything?


https://reviews.llvm.org/D26855