[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