From df8fa6d734417eda61373bbbb025f538c026cd5b Mon Sep 17 00:00:00 2001 From: Simon Pilgrim Date: Fri, 20 Apr 2018 13:12:04 +0000 Subject: [PATCH] [X86][BtVer2] Cleanup some old FIXMEs from the model. NFCI. llvm-svn: 330428 --- llvm/lib/Target/X86/X86ScheduleBtVer2.td | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/llvm/lib/Target/X86/X86ScheduleBtVer2.td b/llvm/lib/Target/X86/X86ScheduleBtVer2.td index 4f5784f..681d8fc 100644 --- a/llvm/lib/Target/X86/X86ScheduleBtVer2.td +++ b/llvm/lib/Target/X86/X86ScheduleBtVer2.td @@ -278,15 +278,13 @@ defm : JWriteResIntPair; def : WriteRes { let Latency = 100; } def : WriteRes { let Latency = 100; } def : WriteRes; + // Nops don't have dependencies, so there's no actual latency, but we set this // to '1' to tell the scheduler that the nop uses an ALU slot for a cycle. def : WriteRes { let Latency = 1; } //////////////////////////////////////////////////////////////////////////////// // Floating point. This covers both scalar and vector operations. -// FIXME: should we bother splitting JFPU pipe + unit stages for fast instructions? -// FIXME: Double precision latencies -// FIXME: SS vs PS latencies //////////////////////////////////////////////////////////////////////////////// def : WriteRes { let Latency = 5; } @@ -311,7 +309,6 @@ defm : JWriteResFpuPair; // NOTE: Doesn //////////////////////////////////////////////////////////////////////////////// // Conversions. -// FIXME: integer pipes //////////////////////////////////////////////////////////////////////////////// defm : JWriteResFpuPair; // Float -> Integer. @@ -344,7 +341,7 @@ def JWriteCVTF2SILd : SchedWriteRes<[JLAGU, JFPU1, JSTC, JFPA, JALU0]> { } def : InstRW<[JWriteCVTF2SILd], (instregex "(V)?CVT(T?)S(D|S)2SI(64)?rm")>; -// FIXME: f+3 ST,LD+STC latency +// FIXME: f+3 ST, LD+STC latency def JWriteCVTSI2F : SchedWriteRes<[JFPU1, JSTC]> { let Latency = 9; let NumMicroOps = 2; -- 2.7.4