This patch improves sched models for SHLD/SHRD* X86 instrs removing unnecessary redefinitions of instr infos. This patch is based on results of D48222.
You should be able to use the X86WriteRes multiclass for all of these (in all models)
Don't leave commented out entries
Can you fix the test changes?
Why not used like the other models?
But I removed
defm WriteShiftDouble : X86SchedWritePair;
Why do we need the comments?
Are you sure? For other CPUs we have WriteSHDrri, WriteSHDrrcl,etc. for all sizes but in Atom it covers 32-bit version only. All other sizes have special redefinitions. To make this clear I put here this comment. Are you sure I have to remove it?