[PATCH] D66424: [X86][Btver2] Fix latency and throughput of CMPXCHG instructions.

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 19 10:47:15 PDT 2019


RKSimon added inline comments.


================
Comment at: test/tools/llvm-mca/X86/BtVer2/resources-cmpxchg.s:48
+# CHECK-NEXT:  6      17    17.00   *      *            lock		cmpxchgl	%ebx, (%rsi)
+# CHECK-NEXT:  6      17    17.00   *      *            lock		cmpxchgq	%rbx, (%rsi)
 
----------------
Only the 8b/16b ops should be in resources-cmpxchg.s, the rest should be in resources-x86_64.s - the non-lock ops are already there. Ideally we'd have coverage for all the other lock ops (ADD, ADC, etc.) but I'm happy to deal with that later.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D66424/new/

https://reviews.llvm.org/D66424





More information about the llvm-commits mailing list