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

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 16 13:41:40 PST 2017


RKSimon 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
----------------
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


https://reviews.llvm.org/D26855





More information about the llvm-commits mailing list