[PATCH] D26855: New unsafe-fp-math implementation for X86 target
Andrew V. Tischenko via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jan 18 01:03:19 PST 2017
avt77 added inline comments.
Comment at: lib/Target/X86/X86InstrInfo.cpp:9523
+/// vmovss xmm1, dword ptr [rip + .LCPI0_0] # xmm1 = mem,zero,zero,zero
+/// vrcpss xmm2, xmm0, xmm0
> 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?
More information about the llvm-commits