[llvm] r318611 - [X86] Fix 80 column violation and remove trailing whitespace. NFC

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Sat Nov 18 17:11:58 PST 2017


Author: ctopper
Date: Sat Nov 18 17:11:58 2017
New Revision: 318611

URL: http://llvm.org/viewvc/llvm-project?rev=318611&view=rev
Log:
[X86] Fix 80 column violation and remove trailing whitespace. NFC

Modified:
    llvm/trunk/lib/Target/X86/X86Subtarget.cpp

Modified: llvm/trunk/lib/Target/X86/X86Subtarget.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86Subtarget.cpp?rev=318611&r1=318610&r2=318611&view=diff
==============================================================================
--- llvm/trunk/lib/Target/X86/X86Subtarget.cpp (original)
+++ llvm/trunk/lib/Target/X86/X86Subtarget.cpp Sat Nov 18 17:11:58 2017
@@ -269,13 +269,14 @@ void X86Subtarget::initSubtargetFeatures
   else if (isTargetDarwin() || isTargetLinux() || isTargetSolaris() ||
            isTargetKFreeBSD() || In64BitMode)
     stackAlignment = 16;
- 
-  // Gather is available since Haswell (AVX2 set). So technically, we can generate Gathers 
-  // on all AVX2 processors. But the overhead on HSW is high. Skylake Client processor has
-  // faster Gathers than HSW and performance is similar to Skylake Server (AVX-512). 
-  // The specified overhead is relative to the Load operation."2" is the number provided 
-  // by Intel architects, This parameter is used for cost estimation of Gather Op and 
-  // comparison with other alternatives.
+
+  // Gather is available since Haswell (AVX2 set). So technically, we can
+  // generate Gathers on all AVX2 processors. But the overhead on HSW is high.
+  // Skylake Client processor has faster Gathers than HSW and performance is
+  // similar to Skylake Server (AVX-512). The specified overhead is relative to
+  // the Load operation. "2" is the number provided by Intel architects. This
+  // parameter is used for cost estimation of Gather Op and comparison with
+  // other alternatives.
   if (X86ProcFamily == IntelSkylake || hasAVX512())
     GatherOverhead = 2;
   if (hasAVX512())




More information about the llvm-commits mailing list