[llvm] [X86] Remove SlowDivide tuning from GRTTuning (PR #84676)

via llvm-commits llvm-commits at lists.llvm.org
Sun Mar 10 06:37:26 PDT 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-backend-x86

Author: Phoebe Wang (phoebewang)

<details>
<summary>Changes</summary>

The DIV32/64 throughput was improved since Goldmont in the Atom architecture. The Alder Lake-E shows similar number too. So we shouldn't add such tunings to Gracemont and later products.

Checked from Agner Fog's table and uops.info.

---
Full diff: https://github.com/llvm/llvm-project/pull/84676.diff


1 Files Affected:

- (modified) llvm/lib/Target/X86/X86.td (-2) 


``````````diff
diff --git a/llvm/lib/Target/X86/X86.td b/llvm/lib/Target/X86/X86.td
index a2a65ce75d6b9a..8367f938c0ddfa 100644
--- a/llvm/lib/Target/X86/X86.td
+++ b/llvm/lib/Target/X86/X86.td
@@ -1237,8 +1237,6 @@ def ProcessorFeatures {
   // Gracemont
   list<SubtargetFeature> GRTTuning = [TuningMacroFusion,
                                       TuningSlow3OpsLEA,
-                                      TuningSlowDivide32,
-                                      TuningSlowDivide64,
                                       TuningFastScalarFSQRT,
                                       TuningFastVectorFSQRT,
                                       TuningFast15ByteNOP,

``````````

</details>


https://github.com/llvm/llvm-project/pull/84676


More information about the llvm-commits mailing list