[PATCH] D50070: [X86] Improved sched models for X86 CMPXCHG* instructions

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added inline comments.


================
Comment at: lib/Target/X86/X86InstrInfo.td:2083
 
-let SchedRW = [WriteALULd, WriteRMW], mayLoad = 1, mayStore = 1,
+let SchedRW = [WriteCMPXCHGLd, WriteRMW], mayLoad = 1, mayStore = 1,
     hasSideEffects = 0 in {
----------------
Wasn't the plan to add a WriteCMPXCHGRMW WriteSequence class?


================
Comment at: lib/Target/X86/X86SchedSandyBridge.td:956
 }
-def: InstRW<[SBWriteResGroup81], (instregex "CMPXCHG(8|16|32|64)rm")>;
+def: InstRW<[SBWriteResGroup81], (instregex "CMPXCHG(8|16)B")>;
 
----------------
craig.topper wrote:
> I didn't notice the regular expression here had changed. The information I gave was for WriteCMPXCHGLd
Why has the regex changed?


https://reviews.llvm.org/D50070