[PATCH] D49310: [llvm-mca][BtVer2] Teach how to identify dependency-breaking idioms.

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added inline comments.


================
Comment at: test/tools/llvm-mca/X86/BtVer2/one-idioms.s:28
 
 # FIXME: their handling is broken in llvm-mca.
 
----------------
You probably wanted to drop this comment.


================
Comment at: tools/llvm-mca/DispatchStage.cpp:110
   // An example of dependency-breaking instruction on X86 is a zero-idiom XOR.
-  if (!Desc.isZeroLatency())
-    for (std::unique_ptr<ReadState> &RS : IS.getUses())
----------------
So it will no longer even consider the sched profile?


https://reviews.llvm.org/D49310