Atom
- No domain switching penalties
Nehalem+
- No penalty on moves
Haswell+
- No penalty on moves / shuffles
Skylake+
- No penality on moves / shuffles / blends
Paths
| Differential D143859
[X86] Adding tuning flags for int <-> fp domain switching penalties; NFC ClosedPublic Authored by goldstein.w.n on Feb 12 2023, 2:14 PM.
Details Summary Atom
Nehalem+
Haswell+
Skylake+
Diff Detail
Event Timeline
Comment Actions LGTM - testing can wait until we start using these (but we will need to have coverage driven by -mattr=+no-bypass-delay as well as specific CPU targets) This revision is now accepted and ready to land.Feb 13 2023, 9:09 AM Comment Actions
They both modify X86.td see L882. goldstein.w.n added a child revision: D144442: [X86] Add tests for replacing `{v}unpck{l|h}pd` -> `{v}shufps`; NFC.Feb 20 2023, 6:11 PM goldstein.w.n removed a child revision: D143787: [X86] Add new pass `X86FixupInstTuning` for fixing up machine-instruction selection.. goldstein.w.n mentioned this in D144442: [X86] Add tests for replacing `{v}unpck{l|h}pd` -> `{v}shufps`; NFC.Feb 26 2023, 10:47 AM goldstein.w.n edited child revisions, added: D144779: [X86] Add tests for replacing `{v}permilps` -> `{v}shufps/{v}pshufd`; NFC; removed: D144442: [X86] Add tests for replacing `{v}unpck{l|h}pd` -> `{v}shufps`; NFC. This revision was landed with ongoing or failed builds.Feb 27 2023, 4:53 PM Closed by commit rGcecaf295898f: Adding tuning flags for int <-> fp domain switching penalties; NFC (authored by goldstein.w.n). · Explain Why This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 496796 llvm/lib/Target/X86/X86.td
llvm/lib/Target/X86/X86Subtarget.h
llvm/lib/Target/X86/X86TargetTransformInfo.h
|
Do you have plans to use all of these domain tuning flags?